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

fix: DELETE periods throws 404 even if the entity exists #72

Open
mozsarmate opened this issue Sep 4, 2024 · 5 comments
Open

fix: DELETE periods throws 404 even if the entity exists #72

mozsarmate opened this issue Sep 4, 2024 · 5 comments
Labels
backend bug Something isn't working NOT PRIO

Comments

@mozsarmate
Copy link
Contributor

No description provided.

@mozsarmate mozsarmate added bug Something isn't working backend PRIO labels Sep 4, 2024
Copy link

linear bot commented Sep 4, 2024

@mozsarmate mozsarmate changed the title fix: DELETE periods throws 404 fix: DELETE periods throws 404 even if the entity exists Sep 4, 2024
@Tschonti Tschonti self-assigned this Sep 5, 2024
@Tschonti
Copy link
Member

Tschonti commented Sep 5, 2024

I couldn't reproduce this. But sometimes the deleted period doesn't disappear from the period list until the page is refreshed

@Tschonti Tschonti removed their assignment Sep 5, 2024
@mozsarmate
Copy link
Contributor Author

I get a 404 each time if the period has at least one applicaiton assigned to it.
I succed to delete the period after i delete all the applications assigned manually

@mozsarmate
Copy link
Contributor Author

maybe we should implement a delete applicaiton button for the admins?? @FearsomeRover

@Tschonti
Copy link
Member

Tschonti commented Sep 6, 2024

or you need to set the onDelete to cascade on the applicationPeriod entity so deleting it will delete all the related applications as well

@mozsarmate mozsarmate added NOT PRIO and removed PRIO labels Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend bug Something isn't working NOT PRIO
Projects
None yet
Development

No branches or pull requests

2 participants