pix
pix copied to clipboard
[TECH] Remove ember-route-action-helper (PIX-5471)
:unicorn: Problème
Décrivez ici le besoin ou l'intention couvert par cette Pull Request.
:robot: Solution
Ajoutez à cet endroit, si nécessaire, des détails concernant la solution technique retenue et mise en oeuvre, des difficultés ou problèmes rencontrés.
:rainbow: Remarques
Reste à faire :
- Ajouter un test sur les challenges
- Supprimer la dépendence
- Vérifier que tout va bien
:100: Pour tester
Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc.
I'm deploying this PR to these urls:
- App (.fr): https://app-pr4740.review.pix.fr
- App (.org): https://app-pr4740.review.pix.org
- Orga: https://orga-pr4740.review.pix.fr
- Certif: https://certif-pr4740.review.pix.fr
- Admin: https://admin-pr4740.review.pix.fr
- API: https://api-pr4740.review.pix.fr/api/
Please check it out!
@MelanieMEB j'ai du code en local qui ajoute des tests 🙈
Review fonc OK
Je n'ai pas réussi à retrouver dans la doc Ember ce que je suis sur le point de vous dire, donc peut-être ont-ils changé de philosophie entre-temps. En tout cas il y a encore quelques temps Ember insistait sur l'application du pattern DDAU (Data Down Action Up) qui faisait en sorte que les appels API ne soient jamais effectués par les components mais par le controller. Du coup je ne suis pas super fan l'implém dans cette PR, mais si ça marche + je n'ai pas réussi à trouver ça dans la doc aujourd'hui = go si ça marche !