[TECH] Mettre à jour Ember en 3.26 sur Pix App (PIX-3081).

[sbedeau]

🦄 Problème

Ember n'est pas à jour sur Pix App

🤖 Solution

Mettre à jour Ember

• ✅ [email protected]
• 🏗️ [email protected]

🌈 Remarques

Plusieurs dépréciations dans la 3.26 :

• https://deprecations.emberjs.com/v3.x/#toc_optional-feature-jquery-integration
• https://deprecations.emberjs.com/v3.x/#toc_implicit-injections ; pour retrouver les stores implicites (donc les fichiers dans lesquels on retrouve this.store mais pas @service store;), on a utilisé la ligne suivante :

grep --files-without-match "@service store;" $(grep --recursive --files-with-matches "this.store" ./mon-pix/app)

grep --files-without-match "service: store;" $(grep --recursive --files-with-matches "this.store" ./mon-pix/app)

• https://deprecations.emberjs.com/v3.x/#toc_this-property-fallback : 19371, a priori les fichiers qui contiennent {{!-- template-lint-disable no-implicit-this --}} sont à nettoyer partout où il manque un this. implicite.

💯 Pour tester

Se balader sur Pix App

[pix-service]

I'm deploying this PR to these urls:

• App (.fr): https://app-pr3382.review.pix.fr
• App (.org): https://app-pr3382.review.pix.org
• Orga: https://orga-pr3382.review.pix.fr
• Certif: https://certif-pr3382.review.pix.fr
• Admin: https://admin-pr3382.review.pix.fr
• API: https://api-pr3382.review.pix.fr/api/

Please check it out!

[jbuget]

Ce ne serait pas mieux de se contenter de monter jusqu'à la version LTS (3.24.5) ?

Autre proposition : monter les autres apps Ember sur la LTS.

Ca permettrait d'attendre que la prochaine LTS (3.28) – prévue initialement la semaine du 09/08 – sorte.

[sbedeau]

Ce ne serait pas mieux de se contenter de monter jusqu'à la version LTS (3.24.5) ?

Autre proposition : monter les autres apps Ember sur la LTS.

Ca permettrait d'attendre que la prochaine LTS (3.28) – prévue initialement la semaine du 09/08 – sorte.

Eventuellement. À noter que 3.25 pour Pix est un non-événement et que 3.26 est douloureuse mais que l'on n'y coupera pas si l'on souhaite aller vers 3.28

J'ai regardé :

• admin : 3.25
• certif : 3.26
• orga : 3.23

[jbuget]

Intéressant. Du coup, je contre-propose de passer toutes les applis à 3.26 avant d'entamer l'upgrade à 3.28.

[jbuget]

3.26 est douloureuse

Qu'est-ce qui est dur ? (remarque : je pourrais regarder moi-même)

[sbedeau]

3.26 est douloureuse

Qu'est-ce qui est dur ? (remarque : je pourrais regarder moi-même)

Pour le moment j'affronte https://deprecations.emberjs.com/v3.x/#toc_implicit-injections qui est partout

Il y a aussi :

• https://deprecations.emberjs.com/v3.x/#toc_optional-feature-jquery-integration qui peut être problématique
• la dépendance ember-collapsible-panel qui est abandonnée et obsolète pour Ember qui durcit l'Octanisation

[sbedeau]

3.26 est douloureuse

Qu'est-ce qui est dur ? (remarque : je pourrais regarder moi-même)

Pour le moment j'affronte deprecations.emberjs.com/v3.x/#toc_implicit-injections qui est partout

Il y a aussi :

• deprecations.emberjs.com/v3.x/#toc_optional-feature-jquery-integration qui peut être problématique
• la dépendance ember-collapsible-panel qui est abandonnée et obsolète pour Ember qui durcit l'Octanisation

Avec du recul ces dépréciations ne devraient pas poser de problèmes puisque tolérées jusqu'à Ember@4.

Comportement étrange constaté : l'appli tourne correctement en fresh start et après l'exécution des tests, l'appli part en banane complètement (app + tests)

[MathieuGilet]

Avec @jonasgrilleres on a ouvert une PR afin de faire une montée de version d'ember-simple-auth en version 4.0.0 (#3524). Cependant, bien qu'il y ait dans cette version un fix censé corriger le problème d'injection du store dans le service session (mainmatter/ember-simple-auth#2302), le problème semble persister malgré tout.

[xav-car]

On met en pause cette PR, il faut faire la migration ember-simple-auth 3.1.0 minimum pour régler un souci de plugin. Celui-ci a engendré quelque régression lors de la 1ère tentative.

On est reparti sur la montée de version de ember-simple-auth 3.1.0 avec @iwazaru, nous pensons avoir trouvé les cas qui n'était pas couverts par rapport aux autres applications. On voit avec la Team Accès si on peut régler ces cas-là avant de repartir sur ce ticket.

Pix Orga et Pix Admin sont en cours de montée de version vers la 3.26.1 (Certif l'était déjà).
Attention Pix-UI est déjà en 3.28.

[xav-car]

ember-simple-auth-oidc nous posera aussi problème, il y a une issue qui remonte le fait que les mixins sont deprecated. Mais le repo n'a pas l'air d'avoir d'activité depuis pas mal de mois 🤒

[clemlatz]

Voici un résumé dans le journal des capitaines de ce qui bloque cette PR et celles qui sont liées :
https://1024pix.atlassian.net/wiki/spaces/DEV/pages/3208839169/2021-10-04+-+Semaine+40#Mont%C3%A9e-de-version-de-Ember-sur-Pix-App

TLDR: c'est entre les mains de la Team Accès

[octo-topi]

Voici un résumé dans le journal des capitaines de ce qui bloque cette PR et celles qui sont liées : 1024pix.atlassian.net/wiki/spaces/DEV/pages/3208839169/2021-10-04+-+Semaine+40#Mont%C3%A9e-de-version-de-Ember-sur-Pix-App

TLDR: c'est entre les mains de la Team Accès

Je taggue la PR Team-accès pour que ça soit plus clair sur cette PR qui a 2 mois
PS: un rapport avec #3555 ?

[octo-topi]

Fermeture car plus d'activité depuis 90 jours