For validation, and doing server side computation must update all associated changed records
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 25/100
Piste de recherche
Commencez par suivre le chemin de gather_records et le comportement d’enregistrement utilisé lors des validations et dans server_methods. Déterminez comment un indicateur pourrait forcer temporairement l’enregistrement de tous les enregistrements associés modifiés, puis vérifiez que les enregistrements ordinaires de l’enregistrement parent conservent le comportement existant.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- JavaScript
- Étoiles
- 538
- Forks
- 41
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de hyperstack-org/hyperstack
-
expose_as_operations method Ouverteenhancement needs doc
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
hyperstack-org/hyperstack#359 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
hyperstack-org/hyperstack#267 ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
hyperstack-org/hyperstack#457 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
hyperstack-org/hyperstack#456 ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 15/100
hyperstack-org/hyperstack#455 · 2 commentaires ·
Toutes les issues de hyperstack-org/hyperstack
Issues similaires
-
bug customer-eng Durable Agents Inngest status: needs triage
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
-
optimization optimization:agents-md-curator
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
githubnext/gh-aw-cao#13475 ·
-
[BUG]: "Clear All" in Settings doesn't clear the saved analysis, old data comes back after reload Ouvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
AOSSIE-Org/OrgExplorer#253 · 1 commentaire ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
oxc-project/oxc#26944 ·
-
ai-observability bug team/ai-observability
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100