-
Notifications
You must be signed in to change notification settings - Fork 5
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
Aurelia 2 #28
Comments
I need to get back to it. Aurelia 2 has this sync mode of change notification which causes issue of state tearing. |
Instead of spreading state across multiple fields, maybe group them all into a single object? if they need together, maybe we need to group them together? |
I can use the batch api if it's available. |
This issue is resolvable here because this is an aurelia plugin. For some other js lib out of aurelia landscape, aurelia 1 can observe any POJO change without any problem. But aurelia 2 might have trouble, user might need to wrap it behind batch api in user's app code. This is main worry I had about aurelia 2's sync observing. |
What do you guys think, does anyone find time to take a look at this in the near future? |
@bigopon is the batch API ready yet? |
@3cp it is for both normal and collection |
Thx, I will have a look. |
Batch api worked well in https://github.com/3cp/aurelia2-dnd v0.3.0. Aurelia 2's reorderable-repeat will take some work as the repeater in Aurelia 2 is totally different from Aurelia v1. |
Any luck? |
I will start this. But don't expect finishing fast, as au2 repeater is totally different. Hope I can make it behaving same on the surface. |
As Aurelia 2 is in beta now, are there any plans to support Aurelia 2?
Thanks for a short feedback.
The text was updated successfully, but these errors were encountered: