[FEATURE] Afficher les bannières dynamiquement.

[MathieuGilet]

🎄 Problème

L'affichage et le contenu des bannières des différentes applications front nécessite de builder l'application.
Cela engendre un délai d'au moins 10 minutes entre la décision de mettre une bannière et le moment auquel la bannière est effectivement visible pour les utilisateurs.

🎁 Proposition

Cette contrainte est liée à l'utilisation de variables d'environnement pour définir le texte et la sévérité des bannières.
Ces variables d'environnement ne sont lues uniquement au moment du build.

Pour pallier à cette contrainte, on décide de stocker les données des bannières à afficher, pour chaque front, dans des clés Redis préfixées par information-banners:.

La sauvegarde dans Redis elle-même est effectuée via Pix Exploit permettant un contrôle fort de ce qui est affiché aux utilisateurs.

Les fronts, lors de leur lancement, appelle une route dédiée (/api/banners/:target) périodiquement afin de pouvoir modifier l'affichage des bannières de manière plus réactive qu'actuellement.

🧦 Remarques

On en a profité pour renommer TemporaryStorage en KeyValueStorage et permettre un stockage sans durée d'expiration.

🎅 Pour tester

Lancer Pix Certif.
Vérifier qu'aucune bannière n'est affichée.

Lancer la commande

scalingo -a pix-api-review-pr11071 run "npm run banner:add -- --target=pix-front-review-pr11071 --severity=information --message_fr=\"Coucou encore\" --message_en=\"Hello again\""

Vérifier qu'une bannière n'est affichée.

Lancer la commande

scalingo -a pix-api-review-pr11071 run "npm run banner:remove -- --target=pix-front-review-pr11071"

Vérifier qu'aucune bannière n'est affichée.

[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)
• Junior
• Admin
• API
• Audit Logger

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

• scalingo front
• scalingo api
• scalingo audit-logger

[bpetetot]

Tests OK et revue OK 👍👍👍👍

• Ajout de plusieurs bannière via le script.
• Affichage des bannières.
• Suppression des bannières via le script.

Suggestion: Mettre le temps de polling par défaut (en local) au moins à une minute, car ça spam pas mal quand on dev en local et ça peut être un peu chiant.

[VincentHardouin]

suggestion(non-bloquante) : Il manque peut-être de pouvoir choisir le type d'alerte à retirer : Si on a une alerte information pour plusieurs mois et qu'on ajoute une alerte warning, là on retire les deux pour ajouter de nouveau celle de type information, ce qui peut être sujet à faire des erreurs ou ne pas la remettre.

question : On parle de Pix Exploit dans la description, du coup est-ce que ces scripts ne sont pas des doublons ? 🤔

[bpetetot]

Je pense que Pix Exploit est uniquement pour la production (pour le moment), les scripts permettent à n'importe qui de réaliser les opérations en local/RA/Integration/Recette