For validation, and doing server side computation must update all associated changed records
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
Línea de trabajo
Comienza rastreando la ruta de gather_records y el comportamiento de guardado utilizado durante las validaciones y en server_methods. Determina cómo un indicador podría forzar temporalmente el guardado de todos los registros relacionados modificados y, a continuación, verifica que los guardados ordinarios del registro padre conserven el comportamiento existente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- JavaScript
- Estrellas
- 538
- Forks
- 41
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de hyperstack-org/hyperstack
-
expose_as_operations method Abiertoenhancement needs doc
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
hyperstack-org/hyperstack#359 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
hyperstack-org/hyperstack#267 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
hyperstack-org/hyperstack#457 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
hyperstack-org/hyperstack#456 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
hyperstack-org/hyperstack#455 · 2 comentarios ·
Todos los issues de hyperstack-org/hyperstack
Issues similares
-
ai-observability bug team/ai-observability
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
AllTheMods/ATM-10-L#19 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
SignalK/n2k-signalk#345 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
JuliaPluto/PlutoPlotly.jl#72 ·