[TECH] Builder pix-admin avec Embroider

[nlepage]

🦄 Problème

Admin est la dernière application qui n'est pas sur embroider.

Avec Brocolli :
Le build échoue sur les erreurs de syntaxe, mais ne le signale pas.
En conséquence, le développeur doit au bout d'un moment relancer le build.

🤖 Proposition

Migrer Admin sur Embroider.

🌈 Remarques

Une 1ère tentative de migration a déjà été effectuée là : #6017

La migration introduit un problème lorsqu'on exécute les tests en mode UI (sur localhost:4202/tests), les expects sur les notifications ne échouent pas lorsque celles-ci sont en auto clear.
En attendant un possible merge de mansona/ember-cli-notifications#364, on pointe sur https://github.com/1024pix/ember-cli-notifications dans lequel l'utilisation de ember-get-config a été supprimée.

💯 Pour tester

Faire de la non régression dans toute l'appli.

Pour les erreurs de build :

• Lancer le build.
• Introduire une erreur de syntaxe.
• Vérifier que le build mentionne cette erreur.
• La corriger.
• Vérifier que le build reprend.

[pix-bot-github]

Une fois les applications déployées, elles seront accessibles via les liens suivants :

• App (.fr)
• App (.org)
• Orga (.fr)
• Orga (.org)
• Certif (.fr)
• Certif (.org)
• Admin
• API
• Pix1D

Les variables d'environnement seront accessibles via les liens suivants :

• scalingo front
• scalingo api

[francois2metz]

On avait ici supprimer ces conditions disabled parce qu'on n'arrivait pas a les faire passer, mais il faudrait les remettre.

[nlepage]

hum ok

[nlepage]

Voir 45984faaab9149780e95df298a218c3a94b03a3f pour le fix

[francois2metz]

Pour les notifications, il y a du code spécifique pour l'env de test:

pix/orga/config/environment.js

Lines 145 to 147 in 7722fa6

	ENV['ember-cli-notifications'] = {
	autoClear: null,
	clearDuration: null,

[francois2metz]

Tant qu'a faire aussi, je pense qu'il faudrait mettre à jour embroider en v3.

[nlepage]

Pour les notifications, il y a du code spécifique pour l'env de test

Une explication et un fix là : mansona/ember-cli-notifications#364

[nlepage]

Tant qu'a faire aussi, je pense qu'il faudrait mettre à jour embroider en v3.

wokay

[yannbertrand]

OK pour le build 👍

[francois2metz]

🌹 🌺 💮 🥀 🌷 🌻 🌸 LGTM 🌸 🌻 🌷 🥀 💮 🌺 🌹