-
Notifications
You must be signed in to change notification settings - Fork 580
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
requires
seemingly adds conditions to all proceeding routes.
#2092
Comments
rawleyfowler
changed the title
Weird behaviour with
'requires' seemingly adds to all routes.
Aug 12, 2023
requires
and routes.
rawleyfowler
changed the title
'requires' seemingly adds to all routes.
Aug 12, 2023
requires
seemingly adds to all proceeding routes.
rawleyfowler
changed the title
Aug 12, 2023
requires
seemingly adds to all proceeding routes.requires
seemingly adds conditions to all proceeding routes.
Might be an idea to provide a full copiable Lite app example. I never use
note sure if that makes a difference. |
@daleif Thats not very ergonomic if I need more than 1 condition |
Seems to fix this. Instead of:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Steps to reproduce the behavior
Expected behavior
I expect that adding the authenticated condition to
/
will not cause it to be added to/login
.Actual behavior
authenticated
is added to both routes. Causing a redirect loop when accessing/
, or/login
.If, however I move the order of the routes around, it works as expected.
Thanks.
The text was updated successfully, but these errors were encountered: