Pat O'Callaghan
Pat O'Callaghan
I believe this will have been a change within Ember Data itself and not model fragments specifically. The recommended pattern is that you use hyphen-case when referencing your models. Take...
@basz I think I'm seeing a similar issue to you. While it's not exactly the same it has the same characteristic where the fragment isn't populated after being `null` initially....
So I've come back to this issue after a long time as we're in the midst of trying get ourselves unblocked to upgrade Ember Data (currently stuck on ED 3.12)....
I've added a potential direction for a fix for this issue in https://github.com/adopted-ember-addons/ember-data-model-fragments/pull/384 but it _seems_ a bit hacky.
@basz I'm not sure if you're issue was exactly the same as mine but could you try out https://github.com/adopted-ember-addons/ember-data-model-fragments/releases/tag/v5.0.0-beta.3 where I shipped my fix to make sure `null` fragments are...
@mike-engel I'm unsure of the correct behaviour here but out of curiosity do you know what happens with regular Ember Data `hasMany` relationships? Do they trigger `did-update` to run?
@dineshrbkumar is this a similar issue to what's described in https://github.com/adopted-ember-addons/ember-data-model-fragments/issues/375? Specifically my comment and the failing test in https://github.com/adopted-ember-addons/ember-data-model-fragments/pull/384? If not, any chance you could create a failing test...
@Gorzas I think this is a good idea. It might be worth doing to get it into the 5.0.0 beta release. It might also be better to use the specific...
@rwjblue I've also just been hit by this issue in our app. `Object.fromEntries` is not polyfilled by `@babel/polyfill`. This is because `@babel/polyfill` (now deprecated) ships with core-js 2.x (2.x is...
Okay thanks, I'll see if I can dig in at some stage this week