feat: use protected routes #136
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Jira board reference:
What does this do?
This PR implements the use of Protected Routes to have separate stacks for authenticated/not authenticated.
It has 3 protected stacks:
Also removes the push routes to '/' when success sign in/sign up, since now the protected guards will automatically change stacks when the isAuthenticated changes.
Why did you do this?
There was a bug where after sign in/sign up you could go back to a login screen by swiping, while you were actually already logged in.
Also, its good to have separate stacks to avoid accessing screens you shouldn't.
Who/what does this impact?
The App navigation (layout), now it starts in the onboarding if it wasn't shown yet, and then if its authenticated then it shows the authenticated stack (starting in the feed), otherwise it shows the not authenticated stack (starting with sign in screen).
How did you test this?
Notes:
Screenshots / Previews
Before:
Simulator.Screen.Recording.-.iPhone.16.Pro.-.2025-10-31.at.10.47.59.mp4
After:
Simulator.Screen.Recording.-.iPhone.16.Pro.-.2025-10-31.at.10.40.52.mp4
🧑🎨 Figma Design:
📱 Android:
📱 iOS: