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] Ne pas récupérer de bailleur social si l'usager a coché non ou je ne sais pas sur logement social #2395

Merged
merged 2 commits into from
Mar 28, 2024

Conversation

sfinx13
Copy link
Collaborator

@sfinx13 sfinx13 commented Mar 26, 2024

Ticket

#2394

Description

Ne pas chercher à récupérer de bailleur si l'usager à cocher non ou je ne sais pas

Changements apportés

  • Méthode privée afin de savoir si le logement est social
  • Prise en compte des valeurs ne-sais-pas et nsp dans les méthode evalBoolean

Pré-requis

Tests

Création de signalement

En tant que Service de secours

  • Je saisis je ne sais pas et je soumet mon signalement
  • Je saisis oui sans saisir de bailleur social (car facultatif) et je soumet mon signalement
  • Je saisis oui avec saisie de bailleur social et je soumet mon signalement
  • Je saisis non et je soumet mon signalement

En tant que Locataire

  • Je saisis non pour le logement social et je soumet mon signalement
  • Je saisis oui pour le logement social avec un bailleur social et je soumet mon signalement

Edition modale bailleur

  • Appliquer les mêmes tests

@sfinx13 sfinx13 force-pushed the bugfix/2394-erreur-bailleur branch from 240a3c9 to d88e29d Compare March 26, 2024 15:26
@sfinx13 sfinx13 force-pushed the bugfix/2394-erreur-bailleur branch from d88e29d to b0e47f3 Compare March 26, 2024 15:48
@sfinx13 sfinx13 changed the title [FO] A l'enregistrement ne pas chercher à récupérer un bailleur si l'usager a coché non ou je ne sais pas [FO] A l'enregistrement ne pas récupérer de bailleur si l'usager a coché non ou je ne sais pas Mar 26, 2024
@sfinx13 sfinx13 marked this pull request as ready for review March 26, 2024 16:04
Copy link

sonarcloud bot commented Mar 26, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarCloud

@sfinx13 sfinx13 changed the title [FO] A l'enregistrement ne pas récupérer de bailleur si l'usager a coché non ou je ne sais pas [FO] Ne pas récupérer de bailleur si l'usager a coché non ou je ne sais pas Mar 26, 2024
@sfinx13 sfinx13 changed the title [FO] Ne pas récupérer de bailleur si l'usager a coché non ou je ne sais pas [FO] Ne pas récupérer de bailleur social si l'usager a coché non ou je ne sais pas sur logement social Mar 26, 2024
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, tests OK sauf le dernier qui n'est pas possible car le champ bailleur est obligatoire

Je saisis oui pour le logement social sans un bailleur social et je soumet mon signalement

@sfinx13
Copy link
Collaborator Author

sfinx13 commented Mar 27, 2024

Relecture OK, tests OK sauf le dernier qui n'est pas possible car le champ bailleur est obligatoire

Je saisis oui pour le logement social sans un bailleur social et je soumet mon signalement

Je supprime le test oui effectivement, merci

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.

Relecture et test OK

@numew numew merged commit 9d075ea into develop Mar 28, 2024
3 checks passed
@hmeneuvrier hmeneuvrier deleted the bugfix/2394-erreur-bailleur branch April 11, 2024 13:17
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

3 participants