-
-
Notifications
You must be signed in to change notification settings - Fork 406
Closed
Labels
Description
With #508 and ultimately removing the ember-data store injection here, the use of the default store👇 is uncommon and in some (many?) cases, error prone. This is further complicated that Ember Data Model class instances don't have a find method, thus causing an assertion.
We should deprecate the default store to
Motivation
Confusion. Comment from an RFC...
BIG fan of this RFC. A few weeks ago I spent half a day trying to figure out why our app was doing an unexpected request, until I stumbled upon this gem of functionality
Notes
This RFC does not intend to remove the default model hook and findModel. Although this is entirely possible to expand this RFC to include this if people so desire. Otherwise, I'd be happy to open a new RFC to accomplish this.
SergeAstapov, NullVoxPopuli, nightire, bertdeblock and sandstrom