loopbackio / loopbackio/loopback-datasource-juggler
Use operation hooks instead of deprecated model hooks
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 282
- Forks
- 366
- Merge medio
- 17 h 4 min
- PR fusionados (30 d)
- 10
Descripción
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
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.
Línea de trabajo
Comienza con la discusión en #824 e inspecciona la ruta de relaciones embebidas alrededor de inst.triggerParent. Compara su comportamiento actual con el enfoque del commit d0cc791bc65e5ee7d2447698539da1d1ca0591ee. Se considera terminado cuando las relaciones embebidas sigan actualizando su elemento padre de forma consistente sin depender de model hooks obsoletos.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript
- Área
- database
- Tipo de issue
- Refactorización
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 25/100