1024pix / pix

Service public d'évaluation et de certification des compétences numériques pour tous.
https://pix.fr
GNU Affero General Public License v3.0
231 stars 52 forks source link

[TECH] Supprimer l'utilisation des "render-modifiers" dans l'admin #9911

Closed La-toile-cosmique closed 2 weeks ago

La-toile-cosmique commented 3 weeks ago

:unicorn: Problème

Depuis l'instauration des composants "Glimmer" l'utilisation des "render-modifiers" est considéré un "anti-pattern"

:robot: Proposition

Supprimer "did-insert" et "did-update" du code de l'interface d'administration.

:100: Pour tester

Tests au vert Tester les fonctionnalités impactées:

:rainbow: Remarques

Il y a eu pas mal de refacto. J'ai remarque que PixCheckbox a un comportement un peu étrange, il faut que la propriete "checked" soit vraie pour que soit pris en compte la valeur "isIndeterminate".

pix-bot-github commented 3 weeks ago

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

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

mcampourcy commented 2 weeks ago

Tests func OK, même si je ne suis pas sûre pour les tubes.

J'ai modifié ça, en espérant que ce soit la bonne fonctionnalité :

Capture d’écran 2024-08-23 à 11 02 41

Et modif du candidat OK ✅

Capture d’écran 2024-08-23 à 11 01 19