LearnNewsExamplesServices
Frontmatter
id6251
titledata.Model: afterSetFields() => take ownership of creating & updating the record class
stateClosed
labels
enhancement
assigneestobiu
createdAtJan 18, 2025, 2:19 PM
updatedAtJan 18, 2025, 2:32 PM
githubUrlhttps://github.com/neomjs/neo/issues/6251
authortobiu
commentsCount0
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtJan 18, 2025, 2:32 PM

data.Model: afterSetFields() => take ownership of creating & updating the record class

Closed v8.6.0 enhancement
tobiu
tobiu commented on Jan 18, 2025, 2:19 PM

from a separation of concerns perspective: a store should be in charge of creating records, the model should be in charge of creating the record class.

  • we can remove the observable mixin again
  • directly connect to the data.RecordFactory instead
tobiu added the enhancement label on Jan 18, 2025, 2:19 PM
tobiu assigned to @tobiu on Jan 18, 2025, 2:19 PM
tobiu referenced in commit d92cde9 - "data.Model: afterSetFields() => take ownership of creating & updating the record class #6251" on Jan 18, 2025, 2:31 PM
tobiu closed this issue on Jan 18, 2025, 2:31 PM
tobiu reopened this issue on Jan 18, 2025, 2:32 PM
tobiu referenced in commit afc9793 - "#6251: removed the static observable class field" on Jan 18, 2025, 2:32 PM
tobiu closed this issue on Jan 18, 2025, 2:32 PM