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

ETQ usager (amélioration) - je ne veux pas de bug d'affichage de message d'information dans la liste des dossiers #10683

Merged
merged 1 commit into from
Aug 8, 2024

Conversation

Benoit-MINT
Copy link
Contributor

@Benoit-MINT Benoit-MINT commented Aug 7, 2024

issue : #10648

Le bug d'affichage se produit lorsque l'attribut "closing_reason" de la procédure est nil, ce qui implique lorsque l'on génère un bloc info (suite à la clôture de la procédure), que aucune condition ne soit remplie.

  1. le fait d'obtenir nil n'est pas un comportement normal : ceci est dû au fait que, pour les procédures publiées, on laisse en choix d'action, soit de "clôturer" la procédure, soit de la "supprimer". Si l'admin choisit de "supprimer", cela ne le fait pas passer par le process de clôture, et donc on se retrouve avec nil dans l'attribut
    Capture d’écran 2024-08-07 à 17 36 59

=> pour éviter au passage la confusion, il est proposé de supprimer l'action "supprimer" laissant ainsi la seule possibilité de passer par "clôturer" lorsque l'admin souhaite mettre un terme à la procédure
Capture d’écran 2024-08-07 à 17 36 31

  1. dans l'hyp où dans la BD on a des procédures qui ont été supprimées de la sorte, cad qui ont l'attribut clossing_reason à nil, on ne peut donc dans dossierss_list conserver les conditions telles quelles.
    => il est proposé de supprimer la condition faisant appel à closing_reason_other? afin que l'on puisse également englober dans ces cas les procédures qui sont à clossing_reason = nil

  2. j'ai profité par ailleurs de cette PR pour homogénéiser les messages d'information présentés à l'usager

@Benoit-MINT Benoit-MINT added the bug label Aug 7, 2024
Copy link

codecov bot commented Aug 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.22%. Comparing base (065000c) to head (67d9ef2).

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #10683   +/-   ##
=======================================
  Coverage   80.22%   80.22%           
=======================================
  Files        1242     1242           
  Lines       26446    26446           
  Branches     4746     4746           
=======================================
  Hits        21217    21217           
  Misses       5229     5229           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Benoit-MINT Benoit-MINT marked this pull request as ready for review August 8, 2024 12:47
@Benoit-MINT
Copy link
Contributor Author

ping @marleneklok pour info

@Benoit-MINT Benoit-MINT added this pull request to the merge queue Aug 8, 2024
Merged via the queue into main with commit 289da2d Aug 8, 2024
18 checks passed
@Benoit-MINT Benoit-MINT deleted the etq-usager-bug-affichage-message-info-dosssier branch August 8, 2024 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Communiqué, ou a ne pas communiqué
Development

Successfully merging this pull request may close these issues.

2 participants