For validation, and doing server side computation must update all associated changed records
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 25/100
Rechercherichtung
Beginne damit, den Pfad von gather_records und das beim Validieren und in server_methods verwendete Speicherverhalten nachzuverfolgen. Ermittle, wie ein Flag erzwingen könnte, dass vorübergehend alle geänderten zugehörigen Datensätze gespeichert werden, und überprüfe anschließend, dass gewöhnliche Speicherungen des übergeordneten Datensatzes das bestehende Verhalten beibehalten.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
In commit 36516765f840eade6a8862682dd5a12774b259cd a fix was made so that ActiveRecord#changed? would accept optionally the attribute that was changing. I.e. Job.changed? means anything in the job changes, vs Job.changed(:name) means did the name attribute change.
Before Job.changed?(:name) was returning the same as Job.changed?
When this fix was made the gather_records method stopped including has_many records that had changed when gathering the parent during a save. This is actually correct behavior according to Rails. However it is a breaking deprecation for some apps BTW.
The bigger problem is that during validations or computations of server_methods we DO want to save (temporarily) ALL related changed records so the computations can use any values that have changed on the client during the server computation.
So we need a flag (perhaps it is the same as the force flag) that can be used to force ALL related changed records to be saved.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 538
- Forks
- 41
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beitragsleitfaden
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus hyperstack-org/hyperstack
-
enhancement needs doc
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
hyperstack-org/hyperstack#359 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
hyperstack-org/hyperstack#267 ·
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
hyperstack-org/hyperstack#457 ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100
hyperstack-org/hyperstack#456 ·
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 15/100
hyperstack-org/hyperstack#455 · 2 Kommentare ·
Alle Issues in hyperstack-org/hyperstack
Ähnliche Issues
-
ai-observability bug team/ai-observability
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
AllTheMods/ATM-10-L#19 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
SignalK/n2k-signalk#345 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
-
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 88/100
JuliaPluto/PlutoPlotly.jl#72 ·