Skip to content

feat (internal): support legacy model behaviors in SchemaRecord legacy mode#9095

Merged
runspired merged 15 commits intomainfrom
schema-record/legacy-features
Nov 11, 2023
Merged

feat (internal): support legacy model behaviors in SchemaRecord legacy mode#9095
runspired merged 15 commits intomainfrom
schema-record/legacy-features

Conversation

@runspired
Copy link
Contributor

@runspired runspired commented Nov 9, 2023

  • id
  • constructor.modelName
  • currentState
  • errors
  • state flags
  • object lifecycle flags
  • serialize
  • _createSnapshot
  • reload
  • changedAttributes
  • rollbackAttributes
  • unloadRecord
  • deleteRecord
  • save
  • belongsTo references
  • hasMany references
  • destroyRecord

Tests

  • id
  • constructor.modelName
  • currentState
  • errors
  • state flags
  • object lifecycle flags
  • serialize
  • _createSnapshot
  • reload
  • changedAttributes
  • rollbackAttributes
  • unloadRecord
  • deleteRecord
  • save
  • [-] belongsTo references (test once legacy belongsTo support is done)
  • [-] hasMany references (test once legacy hasMany support is done)
  • destroyRecord

TBD

  • schema methods on instance
  • schema methods on constructor

Note: only superficially tested. We're largely just copying model code over in a non-diverging manner so as long as the basic mechanics work we should be fine. So for instance we don't need to test every behavior around state flags.

@runspired runspired added 🎯 canary PR is targeting canary (default) 🏷️ feat This PR introduces a new feature labels Nov 9, 2023
@runspired runspired force-pushed the schema-record/legacy-features branch from 1f3eeec to fdcb483 Compare November 10, 2023 01:46
@runspired runspired merged commit 73f7ae6 into main Nov 11, 2023
@delete-merged-branch delete-merged-branch bot deleted the schema-record/legacy-features branch November 11, 2023 00:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎯 canary PR is targeting canary (default) 🏷️ feat This PR introduces a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant