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

[BO - Signalement] Mise à disponibilité des rapports de visite #2314

Merged
merged 7 commits into from
Mar 8, 2024

Conversation

emilschn
Copy link
Collaborator

@emilschn emilschn commented Mar 4, 2024

Ticket

#1983
#2307

Description

Lorsqu'un rapport de visite est saisi ou édité, le fichier de rapport de visite n'apparaissait pas dans les suivis générés automatiquement.
Le rapport de visite n'était pas non plus accessible via la liste des documents de la procédure.

Changements apportés

  • Ajout d'un lien vers le rapport dans les suivis générés lors de la confirmation ou l'édition d'une visite
  • Ajout du rapport de visite dans la partie document des signalements
  • Migration pour modifier les types possibles de photos et documents, pour que ce soit plus parlant

Tests

  • make load-migrations : les documents prennent le bon type
  • Je crée une visite dans le passé en ajoutant un rapport de visite. Il est bien accessible dans le suivi : BO et Fiche usager
  • J'édite une visite déjà effectuée. Idem
  • Les documents sont bien accessibles dans la liste des documents
  • Je ne peux pas éditer le rapport de visite directement dans la liste des documents

@emilschn emilschn changed the title [BO - Signalement] Accessibilité des rapports de visite [BO - Signalement] Mise à disponibilité des rapports de visite Mar 4, 2024
@hmeneuvrier
Copy link
Collaborator

Tout est OK dans les tests.
Je m'aperçois seulement qu'on peut mettre un rapport de visite en pptx, qu'il est bien ajouté et qu'il apparait dans les suivis, mais pas dans la liste de documents.

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.

Souci sur les droit d'accès au doc pour les usager (qui n'est pas propre a cette PR) tous le reste me semble OK

Copy link

sonarcloud bot commented Mar 7, 2024

Quality Gate Passed Quality Gate passed

Issues
8 New issues
0 Accepted issues

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

See analysis details on SonarCloud

Copy link
Collaborator

@sfinx13 sfinx13 left a comment

Choose a reason for hiding this comment

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

Lecture et test 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.

Ok

@numew numew merged commit 568ba3f into develop Mar 8, 2024
3 checks passed
@sfinx13 sfinx13 deleted the fix/1983-rapport-visites branch March 11, 2024 08: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

4 participants