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] Rapport de visite non modifiable #2135

Merged
merged 5 commits into from
Jan 30, 2024

Conversation

numew
Copy link
Collaborator

@numew numew commented Jan 19, 2024

Ticket

#2071

Description

Ajout d'un bouton permettant de supprimer le rapport d'une visite (ce qui permet ensuite d'en uploader un autre via le bouton "Editer le rapport"

Changements apportés

  • Refactoring dans UploadHandlerService pour avoir une fonction deleteFileInBucket

Tests

  • Créer une visite passé sur un signalement, lui ajouter un document rapport, tester sa suppression et l'ajout d'un autre doc de remplacement

@numew numew changed the title Feature/2071 rapport de visite non modifiable [BO - Signalement] Rapport de visite non modifiable Jan 19, 2024
@emilschn
Copy link
Collaborator

Je trouve ça bizarre d'avoir le bouton de suppression directement dans l'affichage.
Au niveau UI, ça aurait plus de sens dans la modale d'édition, non ?
T'as un avis @mathildepoulpux ? :)

@hmeneuvrier
Copy link
Collaborator

Test OK, mais effectivement peut-être plus dans la modale vu que le bouton s'appelle "éditer le rapport"

@mathildepoulpux
Copy link
Collaborator

Je trouve ça bizarre d'avoir le bouton de suppression directement dans l'affichage. Au niveau UI, ça aurait plus de sens dans la modale d'édition, non ? T'as un avis @mathildepoulpux ? :)

Effectivement, ça semple plus logique dans la modale !

target="_blank">Voir le rapport de visite</a>
<ul class="fr-btns-group fr-btns-group--inline-sm fr-btns-group--sm fr-ml-0-5v">
<li>
<a href="{{ asset('_up/'~intervention.files[0].filename)~'/' ~ signalement.uuid }}"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Peut-être que ça vaut le coup de revenir à la version d'avant, notamment avec l'icone dans le bouton ?
Ceci dit, je me rends compte que ça n'a pas l'air de fonctionner actuellement, ça a peut-être sauté avec la MAJ du DSFR sans qu'on ne le remarque...
Si jamais tu peux fouiller pour remettre l'icone, ça pourrait être cool :)

Copy link
Collaborator Author

@numew numew Jan 29, 2024

Choose a reason for hiding this comment

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

Ok, vu que c'est un lien et pas un bouton ca ne fonctionne pas de la même manière, j'ai remis l'icone mais l'affichage n'est pas exactement le même que pour les boutons

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.

J'ai remis un com pour l'affichage du bouton existant

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.

A part la remarque d'Emilien, pour moi tout est ok

Copy link

sonarcloud bot commented Jan 29, 2024

Quality Gate Failed Quality Gate failed

Failed conditions

B Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

idea Catch issues before they fail your Quality Gate with our IDE extension SonarLint SonarLint

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.

Ok lecture et tests

@emilschn emilschn merged commit c900030 into develop Jan 30, 2024
2 of 3 checks passed
@sfinx13 sfinx13 deleted the feature/2071-rapport-de-visite-non-modifiable 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

4 participants