kir-dev / schbody

https://schbody.vercel.app
1 stars 0 forks source link

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

Open mozsarmate opened 1 week ago

linear[bot] commented 1 week ago

KIR-61 fix: DELETE periods throws 404

Tschonti commented 1 week ago

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

mozsarmate commented 1 week ago

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 commented 1 week ago

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

Tschonti commented 1 week ago

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