adopted-ember-addons / adopted-ember-addons/ember-infinity

Move to Immutable Arrays

Offen
#151 2 Kommentare 2 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
JavaScript
Sterne
374
Forks
128
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

A concern I have is that Ember Infinity heavily relies on "pushing objects" into arrays. I don't think this is a great approach - as it relies on Ember's pushObjects API staying consistent - the `_internalModel` stuff is confusing for folks.

Instead I think we should be creating new arrays based on the existing & new data.

@davidgoli also raised some concern about using the store's `query` object in ember infinity. I agree. David - can you share some of your thoughts here?

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.