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

[FO][BO] Recette du nouveau formulaire - Filtre adresse, contraintes de validation dépôt , typo #2186

Merged
merged 7 commits into from
Feb 2, 2024

Conversation

sfinx13
Copy link
Collaborator

@sfinx13 sfinx13 commented Feb 1, 2024

Tickets

#2160
#2170
#2171
#2172
#2163

Description

Recette du nouveau formulaire

Changements apportés

  • Correction d'une faute de frappe dans le dictionnaire
  • Rendre obligatoires les champs "nom" et "occupant" pour tous les profils, sauf le service secours
  • Correction de l'erreur 500 si la superficie n'est pas renseignée pour les profils ties, particuliers, pro et service de secours
  • Rendre le nombre de pièces obligatoire en mode édition
  • Filtrer l'adresse de l’auto-complétion de l'occupant par le code postal saisi
  • Mise à jour de la collection Postman

Pré-requis

Charger la nouvelle collection Postman (j'ai mis tous les profils en POST et en PUT)
L'UUID se charge automatiquement
Faire une requête POST en tant que locataire, puis enchainer avec le PUT en tant que locataire
image

image

Tests

  • Commencer par saisir un code postal sur la page d'accueil et vérifier que l'adresse autocomplète de l'occupant est filtrée
  • Déposer un signalement en tant que tiers (particulier, professionnel, bailleur) et vérifier que le nom et le prénom sont obligatoires ; ils restent non obligatoires pour les services de secours
  • En tant que tiers (particulier, professionnel, bailleur, service secours), ne pas remplir la superficie et vérifier que le signalement est bien déposé
  • Choisissez "n'a pas d'assurance" et vérifiez qu'il n'y a plus de problème de faute de frappe
  • Via la collection Postman pour les profils non testés, vérifier que vous ne pouvez pas déposer de signalement si le nombre de pièces est vide, le nom et prenom est vide pour tous les profils et que la superficie est vide pour le locataire ou le bailleur

@sfinx13 sfinx13 changed the title Bugfix/2171 add control composition logement [FO] [BO] Recette du nouveau formulaire - Filtre, contraintes de validation dépot , typo Feb 1, 2024
@sfinx13 sfinx13 changed the title [FO] [BO] Recette du nouveau formulaire - Filtre, contraintes de validation dépot , typo [FO][BO] Recette du nouveau formulaire - Filtre adresse, contraintes de validation dépôt , typo Feb 1, 2024
Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

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

Lecture ok, tests ok

Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

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

  • Commencer par saisir un code postal sur la page d'accueil et vérifier que l'adresse autocomplète de l'occupant est filtrée
    -> NOK : Si on ne saisie rien (accès direct à la page du dépot) la recherche se fait avec supplément la valeur "null". Je suis pas super convaincu par le fait de faire ca de façon implicite, quel est votre avis sur le fait d'ajouter la valeur du CP par défaut dans le champ de recherche ?

@hmeneuvrier
Copy link
Collaborator

En édition d'un signalement profil tiers particulier, la superficie et le nombre de pièces sont obligatoires, et je trouve ça étrange vu qu'ils ne sont pas obligatoires sur le FO (enfin au moins la superficie)

@sfinx13
Copy link
Collaborator Author

sfinx13 commented Feb 1, 2024

  • Commencer par saisir un code postal sur la page d'accueil et vérifier que l'adresse autocomplète de l'occupant est filtrée
    -> NOK : Si on ne saisie rien (accès direct à la page du dépot) la recherche se fait avec supplément la valeur "null". Je suis pas super convaincu par le fait de faire ca de façon implicite, quel est votre avis sur le fait d'ajouter la valeur du CP par défaut dans le champ de recherche ?

Le supplément null on peut l'enlever
Mettre le code postal sur le champ de recherche, je crains que ça perturbe ou que les utilisateurs ne s'en servent pas, l'efface pour trouver leur adresse.

Copy link

sonarcloud bot commented Feb 1, 2024

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

4 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

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

Relecture OK.
Je laisse @numew merger vu qu'il a aussi fait des retours

@numew numew merged commit d8da23a into develop Feb 2, 2024
3 checks passed
@sfinx13 sfinx13 deleted the bugfix/2171-add-control-composition-logement branch February 13, 2024 21:05
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.

None yet

4 participants