search

liqd / aula

An online platform for political participation in schools in Germany (not in active development)
https://liqd.net
Other
27 stars 5 forks source link

when class is deleted, no notification pops up, and the redirect goes to the deleted class (which is displayed correctly). #1023

Closed fisx closed 7 years ago

fisx commented 7 years ago

the page remains even after reload. i would expect there to be a 40x error because the class shouldn't exist any more. class overview list and start page do not contain the deleted class, as expected.

fisx commented 7 years ago

i could not reproduce this locally, only on demo1.aula.de. race condition issue?

fisx commented 7 years ago

I also notice that classes that do not exist can still be edited. They just have no students and no other content, but they still show up.

fisx commented 7 years ago

oops, solved the original issue: the upgrade script does not update the data files, which include custom.js, which has been fixed since 0.32.0.

fisx commented 7 years ago

I also notice that classes that do not exist can still be edited. They just have no students and no other content, but they still show up.

not a big deal? now that the redirect works, the only way to get to these pages is to enter the url manually, and if that happens very little damage can be done. i am going to close this issue.