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

Validatie Archiefactiedatum voor vernietigen #336

Open
5 of 7 tasks
SilviaAmAm opened this issue Sep 10, 2024 · 1 comment · Fixed by #337
Open
5 of 7 tasks

Validatie Archiefactiedatum voor vernietigen #336

SilviaAmAm opened this issue Sep 10, 2024 · 1 comment · Fixed by #337
Assignees

Comments

@SilviaAmAm
Copy link
Collaborator

SilviaAmAm commented Sep 10, 2024

https://dimpact.atlassian.net/jira/software/c/projects/PZ/boards/258?selectedIssue=PZ-3820

Als record manager
Wil ik niet dat zaken met archiefactiedatum in de toekomst worden vernietigd
Zodat de wet is gerespecteerd

Acceptatie criteria:

  • Implementeer een check die een foutmelding toont als je een vernietigingslijst met zaken met archiefactiedatum in de toekomst probeert te vernietigen. De knop die de record manager gebruikt om de lijst te vernietigen moet disabled zijn en er een melding moet zijn die uitlegt waarom. => Demi design
  • Validatie op het moment van vernietigen (technisch)

To do

Backend

  • Validation on perform destroy endpoint that list has all zaken with archiefactie datum in the past.
  • Update delete_destruction_list_item task to check if zaken can be deleted and if not mark them as failed, with a helpful error message.

PR: #337

Frontend

  • Disable button to start deleting a list based on whether a destruction list item has an archiefactiedatum in the future.
  • Show a message explaining why the button is disabled
  • Visual indication of which zaken cannot be deleted yet.

Design

  • Design for disabling the button "Start Deletion" with message saying that some zaken cannot be deleted yet.
  • Visual indication on the detail page of which zaken cannot be deleted yet?
@SilviaAmAm SilviaAmAm self-assigned this Sep 10, 2024
SilviaAmAm added a commit that referenced this issue Sep 10, 2024
SilviaAmAm added a commit that referenced this issue Sep 10, 2024
svenvandescheur pushed a commit that referenced this issue Sep 12, 2024
@SilviaAmAm
Copy link
Collaborator Author

Only backend done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants