pix icon indicating copy to clipboard operation
pix copied to clipboard

[TECH] Remove ember-route-action-helper (PIX-5471)

Open frinyvonnick opened this issue 2 years ago • 1 comments

: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.

frinyvonnick avatar Aug 03 '22 15:08 frinyvonnick

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!

pix-service avatar Aug 03 '22 15:08 pix-service

@MelanieMEB j'ai du code en local qui ajoute des tests 🙈

frinyvonnick avatar Aug 10 '22 12:08 frinyvonnick

Review fonc OK

VincentHardouin avatar Aug 11 '22 12:08 VincentHardouin

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 !

laura-bergoens avatar Aug 14 '22 06:08 laura-bergoens