Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TECH] Corrige le code pour suivre la règle de lint no-builtin-form-components. #9288

Merged
merged 16 commits into from
Jun 26, 2024

Conversation

frinyvonnick
Copy link
Member

@frinyvonnick frinyvonnick commented Jun 17, 2024

🦄 Problème

Suite à la PR #9270 qui met en place la configuration pour les fichiers gjs sur PixAdmin, il y a deux nouvelles règles de lint que notre code ne respectent pas.

🤖 Proposition

Dans cette PR, on traite la règle no-builtin-form-components en remplaçant les composants builtin par des composants PixUI.

🌈 Remarques

Il y a de nombreuses régressions visuelles suite à ces changements. Nous aurions besoin que toutes les teams concernées participent à l'effort pour les corriger.

J'ai appliqué le code décrit ici pour migrer : https://github.com/ember-template-lint/ember-template-lint/blob/master/docs/rule/no-builtin-form-components.md#migration

💯 Pour tester

Vérifier les écrans impactés par les changements. Nous ne pouvons pas faire confiance seulement à la couverture de test puisqu'il y a des changements graphiques.

Test Accès (RA de Pix Admin)

Formulaire mise à jour détail utilisateur

  • Cliquer sur l'onglet Utlisateurs
  • Faire une recherche avec un prénom (ex: Ayame) puis aller sur la page de détail correspondante
  • Cliquer sur le bouton modifier
  • Vérifier que les validations se font bien et que l'erreur est bien affichée en cas de problème de saisie :
    • pour les champs prénom / nom / identifiant / adresse email, champs non vide
    • pour le prénom / nom / identifiant (limite de 255 caractères)
    • pour l'adresse email, le format de l'adresse
  • Vérifier que, lorsqu'on sélectionne une locale (ex: nl-BE), l'option sélectionnée est bien affichée.
  • Enfin, tester la mise à jour de chaque champ. Rafraîchir la page ensuite et constater qu'il n'y a pas de régressions dessus.

Formulaire mise à jour d'une organisation

  • Cliquer sur l'onglet Organisations
  • Aller sur la page de détail de l'organisation Lycée Joséphine Baker
  • Cliquer sur le bouton modifier
  • Vérifier que les validations se font bien et que l'erreur est bien affichée en cas de problème de saisie :
    • Pour tous les champs input texte, la limite de 255 caractères
    • Pour le champ nom, non vide
    • Pour le champ crédits, un nombre positif
    • Pour le champ lien de la documentation format de l'url
    • Pour les champs Adresse e-mail du DPO + Adresse e-mail d'activation SCO , le format de l'adresse email
  • Enfin, tester la mise à jour de chaque champ. Rafraîchir la page ensuite et constater qu'il n'y a pas de régressions dessus.

@frinyvonnick frinyvonnick self-assigned this Jun 17, 2024
@frinyvonnick frinyvonnick changed the title Tech fix lint no builtin form components [TECH] Corrige le code pour suivre la règle de lint no-builtin-form-components Jun 17, 2024
@frinyvonnick frinyvonnick changed the title [TECH] Corrige le code pour suivre la règle de lint no-builtin-form-components [TECH] Corrige le code pour suivre la règle de lint no-builtin-form-components. Jun 17, 2024
@pix-bot-github
Copy link

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

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

@frinyvonnick frinyvonnick force-pushed the tech-fix-lint-no-builtin-form-components branch 2 times, most recently from 552baa8 to 5fae55e Compare June 17, 2024 14:02
Base automatically changed from tech-setup-gjs-on-admin to dev June 17, 2024 14:19
@frinyvonnick frinyvonnick force-pushed the tech-fix-lint-no-builtin-form-components branch from 5fae55e to 35d73ca Compare June 17, 2024 15:23
@AndreiaPena AndreiaPena marked this pull request as ready for review June 17, 2024 15:34
@yannbertrand yannbertrand force-pushed the tech-fix-lint-no-builtin-form-components branch from 90e77a2 to 7db14b8 Compare June 18, 2024 08:03
Copy link
Member

@yannbertrand yannbertrand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK DevComp

@AndreiaPena AndreiaPena force-pushed the tech-fix-lint-no-builtin-form-components branch from e10f9a0 to 18914da Compare June 18, 2024 08:28
Copy link
Contributor

@aceol aceol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je ne connaissais pas cette syntaxe, interessant!
{{on "input" (pick "target.value" (set this "form.email"))}}

@matthiasferraina matthiasferraina force-pushed the tech-fix-lint-no-builtin-form-components branch 2 times, most recently from cb3a6a1 to a84f891 Compare June 18, 2024 13:49
@@ -40,6 +41,7 @@
</PixInput>

<PixInput
placeholder={{t "pages.login.fields.password.label"}}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@matthiasferraina Les placeholders sont pas franchement nécessaire, si ? 😅

Capture d’écran 2024-06-18 à 16 30 20

Copy link
Member

@AndreiaPena AndreiaPena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok pour certif :)

@er-lim er-lim force-pushed the tech-fix-lint-no-builtin-form-components branch 2 times, most recently from aa32cb7 to 948029e Compare June 24, 2024 13:32
@machestla
Copy link
Contributor

Ok pour Prescription

Copy link
Contributor

@igorissen igorissen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pix-service-auto-merge pix-service-auto-merge force-pushed the tech-fix-lint-no-builtin-form-components branch from 7fa7bc0 to 12cfd62 Compare June 26, 2024 11:43
@pix-service-auto-merge pix-service-auto-merge merged commit 07ff0a2 into dev Jun 26, 2024
6 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the tech-fix-lint-no-builtin-form-components branch June 26, 2024 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-team Toutes les équipes de dev Func Review OK PO validated functionally the PR 🚀 Ready to Merge Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants