-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Backport Train for LTS #7805
Backport Train for LTS #7805
Conversation
A big thank you to you @runspired, @snewcomer @andreyfel for this hard work. |
Hi all—my team is waiting for this PR to upgrade to 3.28, since we need the fix implemented in #7771. Is anyone available to look at the failing check and merge this? Thank you! |
@allison-strandberg Thanks for the reminder! Seems like we could cherry pick that single commit from the linked PR to 3.28. I can try and get to that tonight. |
@allison-strandberg 3.28.8 released! |
* deactivate broken partner tests * feat: autotracking for reference id access (#7796) * feat: autotracking for reference id access * ensure references are torn down * fix build * add dep * add to deps * fix invalid json:api support and add valid json:api support * autotracking tests and cleanup * fix test failure, add comment * skip tests when feature not available * update test and fix lid reflection (#7800) * update test and fix lid reflection * remove debugger * fix ff off branch * add test and fix push of duplicate identifiers to a relationship (#7801) * add test + fix for chained async has many (#7691) * [bugfix]: fix for chained async has many * add fix and update tests * remove console.logs * make work with flags off * fix test for lts Co-authored-by: Chris Thoburn <[email protected]> * Fix: assign unknown properties in init after initialization is finished to ensure proper setup timing (#7771) * Add failing test case which illustrates the createRecord bug createRecord crashes when a setter which sets an attribute is involved in the createRecord. * update test location and add fix Co-authored-by: Chris Thoburn <[email protected]> * fix: A(PromiseManyArray) should have no-effect (#7802) Co-authored-by: Sylvain Mina <[email protected]> Co-authored-by: Andrey Fel <[email protected]>
7945b5c
to
b445997
Compare
deactivate broken partner tests
feat: autotracking for reference id access (feat: autotracking for reference id access #7796)
feat: autotracking for reference id access
ensure references are torn down
fix build
add dep
add to deps
fix invalid json:api support and add valid json:api support
autotracking tests and cleanup
fix test failure, add comment
skip tests when feature not available
update test and fix lid reflection (update test and fix lid reflection #7800)
update test and fix lid reflection
remove debugger
fix ff off branch
add test and fix push of duplicate identifiers to a relationship (add test and fix push of duplicate identifiers to a relationship #7801)
add test + fix for chained async has many (add test + fix for chained async has many #7691)
[bugfix]: fix for chained async has many
add fix and update tests
remove console.logs
make work with flags off
fix test for lts
Co-authored-by: Chris Thoburn [email protected]
Fix: assign unknown properties in init after initialization is finished to ensure proper setup timing (Fix: assign unknown properties in init after initialization is finished to ensure proper setup timing #7771)
Add failing test case which illustrates the createRecord bug
createRecord crashes when a setter which sets an attribute is involved
in the createRecord.
Co-authored-by: Chris Thoburn [email protected]
Co-authored-by: Sylvain Mina [email protected]
Co-authored-by: Andrey Fel [email protected]