Remove unnecessary redirects and remove temp event stuff. #183

G-tix · 2023-08-09T17:12:30Z

G-tix commented

2023-08-09 17:12:30 +00:00

Removing 400 redirects as this confuses users (including me lol). Also removed some event stuff which seems redundant.

G-tix added 2 commits 2023-08-09 17:12:31 +00:00

12b0d822af Remove unnecessary 400 redirects

The redirects are confusing people (including me lol).

0ffc2e6228 Remove temp event stuff

If still needed, please ignore.

DrTransmisia commented

2023-08-09 17:17:42 +00:00

400 is a bad request [1] after all 🤔

[1] https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/400

400 is a bad request [1] after all 🤔 [1] https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/400

G-tix commented

2023-08-09 17:51:26 +00:00

Poster

@DrTransmisia Yeah, but now you get redirected to a whole different page instead of just an error message like line 794 ( return redirect("/settings/personal?error=Not a valid MP3 file!") ).

This error message is, imo, way easier to understand than a redirect.

@DrTransmisia Yeah, but now you get redirected to a whole different page instead of just an error message like line 794 ( return redirect("/settings/personal?error=Not a valid MP3 file!") ). This error message is, imo, way easier to understand than a redirect.

bad.PNG

7.3 KiB

good.PNG

2.3 KiB