-
Notifications
You must be signed in to change notification settings - Fork 28
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
post.async.get('comment') #30
Comments
Thanks for the message :D
|
You're right. Could you open a PR to fix these things and give better instructions in the README?
I see what you're saying. I personally never needed it, so I ended up never implementing it. Would you be interested in working in a patch to include that?
Totally agree. Let's discuss that in #33.
The thing here is that it returns a stream, which means that say you have 10 records offline and 5 online. You start showing the 10 records right away and then the next 5 are shown a second later. I've been thinking about changing that to a promise, but we'd need to put more thought on it. |
Hi, getting into ember-sync and this is a great work thanks for sharing.
Different points below more or less linked, so I prefer to not open several github issues:
npmjs.com repo not up to date? It may be obvious for many people to check the version when they do 'npm install ember-sync' but I didn't and I spent some hours debugging the same things you have dealt with these last months.. (I am discovering npm, bower etc. so this may be a very naive point).
Doc: my understanding is that if you don't create / save or find a post Record through emberAsync, you will not be able to call post.emberAsync on it later on:
=> this may be clearer in the docs?
When finding a record through this.emberSync.findQuery, the result's relationships don't have the method 'emberSync', so not possible to save them to both stores etc. (in my case, I change one relationship in a form but can not properly save it after).
I think I see a trick to solve that (using this.emberSync.onRecordAdded(inst,inst.get('constructor.typeKey'))), but at the very least this is not in the docs.
=> It would be nice to see emberSyncing relationships possible in a future implementation, probably this would look something like "post.async.get('comment')"?
It would be nice to be able to not fetch from the server if not needed, for example if we know that the records, if they exist, are more recent locally (the user is the author of these records).
Admittedly we could use the offline store directly for this, but we could want to post the records both online and offline later on, after having fetched them and if not initialized properly with emberSync this is not possible.
=> So overall, ideally here would be more options for fetching records, like: 1) fetch only if they don't exist in the offline store [ 2) fetch only if they are more recent on the server (sending an updated_at in the request) etc. ]. 1) would really help in using emberSync for caching data locally.
=> A dirty solution in Find records from offline store, then from online store only if not found #33
Small things:
Many thanks for the excellent work,
L
The text was updated successfully, but these errors were encountered: