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

[Dépôt de besoin] Simplification du formulaire en supprimant les informations de contact pour les utilisateurs connectés #916

Merged
merged 5 commits into from
Sep 26, 2023

Conversation

SebastienReuiller
Copy link
Contributor

@SebastienReuiller SebastienReuiller commented Sep 20, 2023

Quoi ?

Suppression des champs liés au contact (prénom, nom, e-mail, téléphone) quand l'utilisateur est connecté

Pourquoi ?

Pour rendre le formulaire plus simple à remplir.

Comment ?

Les champs sont automatiquement renseignés avec les informations du contact authentifié.

Capture d'écran

Étape de contact pour un utilisateur non connecté

Étape de contact pour un utilisateur non connecté

Étape de contact pour un utilisateur connecté
Étape de contact pour un utilisateur connecté

@SebastienReuiller SebastienReuiller self-assigned this Sep 20, 2023
@SebastienReuiller SebastienReuiller marked this pull request as ready for review September 25, 2023 14:50
Copy link
Contributor

@raphodn raphodn left a comment

Choose a reason for hiding this comment

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

LGTM !

)
else:
setattr(self.instance, attribute, tender_dict.get(attribute))
match attribute:
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, le match n'est dispo que depuis la 3.10. Bon même si là, c'est juste un peu plus lisible, mais on ne tire pas vraiment profit des possibilités de cette nouvelle structure. Mais ça va venir 😉

@SebastienReuiller SebastienReuiller merged commit 8274b6c into master Sep 26, 2023
4 checks passed
@SebastienReuiller SebastienReuiller deleted the sreuiller/tender-fill-name branch September 26, 2023 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants