Skip to content
This repository has been archived by the owner on Oct 22, 2024. It is now read-only.

[Fix] Correction de la gestion des labels nationaux restreints #450

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

jbuget
Copy link
Contributor

@jbuget jbuget commented Sep 12, 2024

🍣 Contexte / problème

Des bugs ont été constatés lors de la recette du ticket :
https://trello.com/c/CVlvfhYh/153-ajouter-label-ft-sur-structures

🦄 Solution

La gestion de ces options de libellés est assez particulière et avancée.

La solution proposée par cette PR consiste à ajouter une propriété – fixedItemValues – au composant de sélection <SimpleAutocomplete/> pour gérer des "options fixes".

Une option fixe ne peut être ni sélectionnée, ni retirée (ex : label "France Travail" pour une agence France Travail)

@jbuget jbuget force-pushed the fix/label-france-travail branch 2 times, most recently from 3c69b18 to ba54362 Compare September 12, 2024 07:58
ikarius
ikarius previously approved these changes Sep 12, 2024
Copy link
Contributor

@ikarius ikarius left a comment

Choose a reason for hiding this comment

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

LGTM
juste un debug a retirer je pense, mais tout est très clair
A toi de voir si tu veux borner par un test ou pas

@jbuget
Copy link
Contributor Author

jbuget commented Sep 12, 2024

Merci pour la relecture et bien vu pour le {@debug} qui traînait 🙏

Tout l'outillage de tests de composants côté front est embarqué dans une autre PR.

Je passe mon tour pour celle-ci, pour le moment, car ça mériterait d'y consacrer beaucoup trop d'efforts par rapport à l'existant :

  • le composant fonctionne en prod depuis 3 ans sans tests, il peut survivre quelques semaines de plus ;
  • il est l'un des plus consultés / utilisés → on va très vite se rendre compte d'un souci ;
  • il n'est absolument pas trivial à tester, surtout qu'on manque encore d'outillage de test de composants côté front)

Je viens de valider les comportements utilisateur sur mon poste avec Chloé. Je pense et je suis chaud pour qu'on l'envoie en recette.

@jbuget jbuget enabled auto-merge September 12, 2024 09:06
@jbuget jbuget merged commit a674f99 into main Sep 12, 2024
7 checks passed
@jbuget jbuget deleted the fix/label-france-travail branch September 12, 2024 14:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants