search

kurko / ember-sync

MIT License
282 stars 28 forks source link

turn into an ember-cli addon #1

Closed jakecraige closed 10 years ago

jakecraige commented 10 years ago

@kurko after merging this you would also need to publish to NPM for the readme to be accurate.

Adds the import and default EmberSync initializer

kurko commented 10 years ago

Looks good! Next time, please open separate PRs for each feature/change :)

EmergentBehavior commented 10 years ago

Yes I tried installing with npm and it said the package was not found. Probably should update your Readme until it's posted.

kurko commented 10 years ago

Yep. I'm doing that as soon as I get home. — Sent from Mailbox

On Wed, Jul 2, 2014 at 10:22 PM, Josh Brandoff notifications@github.com wrote:

Yes I tried installing with npm and it said the package was not found. Probably should update your Readme until it's posted.

Reply to this email directly or view it on GitHub: https://github.com/kurko/ember-sync/pull/1#issuecomment-47856965

kurko commented 10 years ago

@EmergentBehavior bumped up version to 0.1.1 and published to npm. You can try that now. Let us know how's it working for you :)

EmergentBehavior commented 10 years ago

Am just playing with it for now. Right now I'm just using your indexeddb adapter for a project on storj (metadisk) but later i will use the full sync. Great repo though! A lot of people are looking for something like this. @EmberSherpa pointed me your way.

kurko commented 10 years ago

@EmergentBehavior thanks! I extracted this from an app I have and my goal is to gather feedback from people that are going to try it with different use cases, which will most likely show us where Ember Sync might be broken so we can fix it.

If you need a hand, I'm online most of the day at the emberjs channel at Freenode as kurko.

EmergentBehavior commented 10 years ago

Much appreciated. I'm utropics.