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

[Signalement] Modification de la question de préavis de départ #2196

Merged
merged 4 commits into from
Feb 8, 2024

Conversation

emilschn
Copy link
Collaborator

@emilschn emilschn commented Feb 1, 2024

Ticket

#2164

Description

L'information sur le préavis de départ n'était pas obtenue avec la même précision selon les profils.
La question est posée ailleurs, si le déclarant sait si l'occupant a prévu de quitter les lieux.
Dans le BO, elle change de bloc.

Tests

  • FO : En tant que locataire, si j'ai prévu de partir, on me demande si j'ai déposé un préavis de départ
  • BO : On retrouve l'information dans le bloc Situation du foyer, et on peut l'éditer
  • FO : En tant que tiers, si je sais que l'occupant souhaite partir, on me demande si il a déposé un préavis de départ
  • BO : On retrouve l'information dans le bloc Situation du foyer, et on peut l'éditer

@@ -101,6 +102,18 @@ public function setTravailleurSocialQuitteLogement(?string $travailleurSocialQui
return $this;
}

public function getTravailleurSocialPreavisDepart(bool $raw = true): ?string
{
return (!$raw && 'nsp' === $this->travailleurSocialPreavisDepart) ? 'Ne sait pas' : $this->travailleurSocialPreavisDepart;
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

reprise du même principe utilisé par Numa dans sa PR #2194

{% elseif signalement.situationFoyer.travailleurSocialPreavisDepart is same as 'non' %}
{{ static_picto_no|raw }}
{% else %}
{{signalement.situationFoyer.travailleurSocialPreavisDepart(false)}}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Reprise du principe utilisé par Numa dans sa PR #2194

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.

Tests et relecture OK
Veto sur un merge de celle ci avant la #2194

@hmeneuvrier
Copy link
Collaborator

Quelques commentaires, tests pas encore faits

@emilschn
Copy link
Collaborator Author

emilschn commented Feb 5, 2024

@hmeneuvrier retours traités
pas mal de changements @numew si tu veux vérifier

tableau mis à jour ligne 74

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 et tests OK, je pourrai merger quand celle de Numa sera mergée

Copy link

sonarcloud bot commented Feb 8, 2024

Quality Gate Passed Quality Gate passed

Issues
2 New issues

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

See analysis details on SonarCloud

@hmeneuvrier hmeneuvrier merged commit 5f50bec into develop Feb 8, 2024
3 checks passed
@sfinx13 sfinx13 deleted the feature/2164-preavis-depart branch February 13, 2024 21:09
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