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.

Sign up for GitHub

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

Jump to bottom

[Utilisateurs] nouveau type "Particulier" #1002

Merged
merged 4 commits into from
Dec 6, 2023
Merged

[Utilisateurs] nouveau type "Particulier" #1002

merged 4 commits into from
Dec 6, 2023

Conversation

raphodn
Copy link
Contributor

@raphodn raphodn commented Dec 4, 2023

Quoi ?

à l'inscription, pouvoir choisir un nouveau type d'utilisateur : "Particulier" (INDIVIDUAL dans le code)

@raphodn raphodn self-assigned this Dec 4, 2023
@raphodn raphodn added the recette-jetable Déclenche une recette jetable sur CC label Dec 4, 2023
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Dec 4, 2023

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

SebastienReuiller
SebastienReuiller requested changes Dec 6, 2023
View reviewed changes
Copy link
Contributor

@SebastienReuiller SebastienReuiller left a comment

Choose a reason for hiding this comment

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

@raphodn , je ne parviens pas à valider sur la recette jetable. Je pense que ça ne suffit pas d'ajouter un d-none sur infoStructuresFieldset, il doit y avoir des champs requis dedans et mon navigateur doit bloquer le submit pour ça, il voit un champs requis que je ne vois pas 😅.

@raphodn raphodn force-pushed the raphodn/user-type-individual branch from eb5c698 to b81814d Compare December 6, 2023 14:28
@raphodn
Copy link
Contributor Author

raphodn commented Dec 6, 2023

@raphodn , je ne parviens pas à valider sur la recette jetable. Je pense que ça ne suffit pas d'ajouter un d-none sur infoStructuresFieldset, il doit y avoir des champs requis dedans et mon navigateur doit bloquer le submit pour ça, il voit un champs requis que je ne vois pas 😅.

merci d'avoir testé ! en effet y'avait le champ company_name qui restait en required. J'ai corrigé, et testé chaque type d'utilisateur pour m'assurer que cela fonctionnait bien après 🙌

@raphodn raphodn merged commit 06bb27d into master Dec 6, 2023
4 checks passed
@raphodn raphodn deleted the raphodn/user-type-individual branch December 6, 2023 14:40
@raphodn raphodn mentioned this pull request Dec 7, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SebastienReuiller SebastienReuiller SebastienReuiller requested changes

@madjid-asa madjid-asa Awaiting requested review from madjid-asa

Assignees

@raphodn raphodn

Labels
recette-jetable Déclenche une recette jetable sur CC
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@raphodn @SebastienReuiller