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

Merge into Ember Data #52

Closed
sandstrom opened this issue Sep 27, 2018 · 3 comments
Closed

Merge into Ember Data #52

sandstrom opened this issue Sep 27, 2018 · 3 comments

Comments

@sandstrom
Copy link

sandstrom commented Sep 27, 2018

Has there been any discussions with the Ember Data team about moving this functionality into Ember Data?

Basically, to use this addon as the starting point for an RFC that builds on emberjs/rfcs#21, but probably supersedes it in a few ways.

It would be very useful! 🎈

@danielspaniel
Copy link
Owner

This could happen one day @sandstrom , but I am not really pushing for or against it. Just sitting on the fence waiting for the record data things to happen and waiting for ED to evolve a bit this year. People are aware of the need to dirty track relationships ( for sure ) so if you want to hustle that process along, your more than welcome. But I not sure if it is a big priority for ED team.

@sandstrom
Copy link
Author

sandstrom commented Sep 27, 2018

Makes sense, the record data changes seems to be moving along at a good pace (emberjs/data#5618). Better to help bringing that into stable before any work on integrating better dirty tracking.

Awesome addon! 🏅 Perhaps it may grow into ED at some point.

(feel free to either close this or leave open for future discussion)

@danielspaniel
Copy link
Owner

thanks for the good vibes @sandstrom .. keeps me => 🕺

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

No branches or pull requests

2 participants