-
-
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
[ON_HOLD] feat: add additional state-machine states to handle unload/loading edge cases #7475
Conversation
Asset Size Report for 9ffe5bf IE11 Builds 🛑 The size of the library EmberData has increased by +385.0 B (+31.0 B compressed) which exceeds the failure threshold of 75 bytes.WarningsChangeset
Full Asset Analysis (IE11)
Modern Builds 🛑 The size of the library EmberData has increased by +381.0 B (+139.0 B compressed) which exceeds the failure threshold of 75 bytes.WarningsChangeset
Full Asset Analysis (Modern)
Modern Builds (No Rollup) 🛑 The size of the library EmberData has increased by +388.0 B (+78.0 B compressed) which exceeds the failure threshold of 75 bytes.WarningsChangeset
Full Asset Analysis (Modern)
|
Closing in favor of the state machine refactor in #7258 |
resolves emberjs/rfcs#358
resolves emberjs/rfcs#359
While working on #7470 and #7471 I noticed that we don't properly inform RecordData today whether the teardown of relationships is occurring for a new record or not. This is primarily because the context is lost. It's also currently impossible to tell internally if an empty record is empty because it was unloaded after a persisted deletion, has never been fetched, or has been unloaded but retained. This resolves this which will allow us to do nicer things internally in many locations.
We may desire to RFC these, though in most ways this is additive backfill to fix existing bugs with the state machine. It's unclear whether new substates which result in new
currentState.stateName
strings require RFC or not.I scanned ember-observer and no actively maintained addons will be affected (two that were never published and haven't been touched in years would have had a marginal migration to make).