-
Notifications
You must be signed in to change notification settings - Fork 2
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
Improve data sync #190
Improve data sync #190
Conversation
Just native Promise will do.
Throws a linter warning, now that it's not an RSVP promise anymore.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good. Just had some super minor comment.
Tests worked also as promised.
Great work! 👍
@@ -2,12 +2,10 @@ import Component from '@ember/component'; | |||
import { inject as service } from '@ember/service'; | |||
|
|||
export default Component.extend({ | |||
tagName: '', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since Octane, components don't have a tag like div
by default anymore. So this is not necessary.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You mean Glimmer components. But this one hasn't been converted yet.
@galfert Thanks! |
👍 |
This introduces ember-concurrency for managing concurrent tasks, as well as task queues, etc.. The new changes improve the fetching of data in several ways:
After merging this one, we should soon improve the dashboard further by not showing all contributions in the list by default, and also asking the user before fetching the entire history. When we have UI for that, we can then also show sync progress somewhere.