-
-
Notifications
You must be signed in to change notification settings - Fork 429
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
Redirect to last page after login #788
Comments
Hi @moshfeu, I am not sure if this approach would be feasible or not, but once we get the Unauthorised response from the server, before redirecting can't we store the URL of the current page in Let me know your thoughts on this. |
Hi @nikhils4! I'd suggest to use the find-a-mentor/src/utils/auth.js Lines 25 to 27 in a1ad19a
|
@nikhils4 do you want to take this? |
Sorry for not getting back on this, had a crazy last week and would remain unavailable until next week, later I can pick this up, if that's okay. |
No worries! |
Thanks a lot, for sure once available will pick this up :) |
Hi, I have forked the repo and have it running. I would like to work on this issue but would like clarification on the "back office". Is this a space for mentors? |
Actually, I see the issue. |
Great :) Thank you |
@moshfeu Where is isMentorIntent being inputed into the login function? It's not called in the LoginNavigation. I would like to add a from parameter as well, but I'm not sure I can do that without messing up other code. login = isMentorIntent => { |
Nevermind, I figured it out finally. Now I am dealing with the same issue logging out of the backend. It needs to be sent to the front end to logout or else it just keep logging back in. |
Done by #830 👏 |
Currently if a user try to access back office while they are not logged in, they redirected to the home page.
The expected result:
Redirect to login and remember the page they redirected from. After successful login, they should be redirected to this page
The text was updated successfully, but these errors were encountered: