search

memaar92 / transcendence

MIT License
3 stars 0 forks source link

56-no-redirect-when-2fa-missing #58

Closed wrottger closed 2 weeks ago

memaar92 commented 2 weeks ago

Kann ich vermutlich morgen genauer testen. Aber wie sind denn meine file commits hier gelandet? Die waren eigentlich nur auf main, weil dort das Email-Banner Bild gehostet wird. Die brauchen eigentlich nicht auf dev

wrottger commented 2 weeks ago

Kann ich vermutlich morgen genauer testen. Aber wie sind denn meine file commits hier gelandet? Die waren eigentlich nur auf main, weil dort das Email-Banner Bild gehostet wird. Die brauchen eigentlich nicht auf dev

Ich weiss auch nicht wie die hier gelandet sind :)

wrottger commented 2 weeks ago

@wrottger replace the (!request.ok) so it doesn't catch 404 errors

memaar92 commented 2 weeks ago

@wrottger bitte nochmal die geaenderte Logik pruefen und vor geupdateten PR selbst testen. Wenn man (!response.ok) (ich glaub du meintest das; ein "!request.ok" hab ich in dem file nicht gefunden) und das zugehoerige else statement rausnimmt, kommt man zwar nun zum registrieren. Allerdings kommt man von dort nun nicht weiter.