Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Initialized Bower #23

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

derekbarnhart
Copy link

Just ran bower init so the repo can be used via the bower cli tool.

@danielkcz
Copy link

May I ask why it hasn't been pushed into Bower yet ? This seems like very useful utility

@derekbarnhart
Copy link
Author

I would have done it myself but I felt it would be better if the repo owner registered the official repo in the Bower registry. Hence the pull request.

@danielkcz
Copy link

Sure, I agree. However it doesn't seem that owner cares about this anymore. Maybe if you would be willing take over the maintenance ? I am at least glad, that Bower can use github repo directly too.

@derekbarnhart
Copy link
Author

Have you seen the Amplify.js store module? http://amplifyjs.com/api/store/ It might be a better solution since persist-js seems to be growing stale.

@danielkcz
Copy link

That looks great, didn't knew about that. However it doesn't use cookies as the fallback solution. I would like to have that in there.

@derekbarnhart
Copy link
Author

Ah that is a good point. I think amplify makes it fairly trivial to implement your own storage type but I haven't really looked into it. When I get a spare moment I will take a look at that. Should it prove to be too much of a hassle to extend amplify and jeremydurham doesn't object I would definitely consider assuming maintenance of the project.

@danielkcz
Copy link

Well here we are again after 7 months, @derekbarnhart. Seems that @jeremydurham really lost all interestest in maintaining this and since I didn't found viable alternative yet, it would be really nice to have it in the bower. How are you feeling about it ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants