loopbackio / loopbackio/loopback-datasource-juggler
Use operation hooks instead of deprecated model hooks
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 282
- Forks
- 366
- Merge moyen
- 17 h 4 min
- PR mergées (30 j)
- 10
Description
See the discussion in #824. We were not able to remove deprecated model hooks because embedded relations are still relying on them.
There is a feature for
inst.triggerParentin embedsMany that needs more consideration and possibly a rewrite for embedded relations to continue work. It is for updating the parent model if any changes are made to the relation. This part needs a redesign and rewrite to consider the different cases and to make it all consistent.
We should rewrite inst.triggerParent to not depend on deprecated hooks, for example using the approach outlined in https://github.com/strongloop/loopback-datasource-juggler/commit/d0cc791bc65e5ee7d2447698539da1d1ca0591ee
cc @jannyHou @Amir-61 @superkhau
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.
Piste de recherche
Commencez par la discussion dans #824 et examinez le chemin des relations intégrées autour de inst.triggerParent. Comparez son comportement actuel avec l’approche du commit d0cc791bc65e5ee7d2447698539da1d1ca0591ee. Le travail est terminé lorsque les relations intégrées continuent de mettre à jour leur parent de manière cohérente sans dépendre de model hooks obsolètes.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript
- Domaine
- database
- Type d'issue
- Refactorisation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 25/100