talent-connect / connect

Mentor-mentee and jobseeker-company matchmaking platform used by ReDI School of Digital Integration, in Berlin, Munich and Düsseldorf, Germany.
https://connect.berlin.redi-school.org
19 stars 8 forks source link

feat(con/tp): direct to intended page after login #838

Closed ericbolikowski closed 6 months ago

ericbolikowski commented 6 months ago

What Github issue does this PR relate to? Insert link.

824

What should the reviewer know?

This PR ensures a user experiences when trying to access a CON/TP page requiring login but they aren't logged in:

  1. Suppose user tries to access /app/browse, but isn't logged in
  2. User redirected to /front/login?goto=/app/browse (the /app/browse is actually urlencoded so it actually becomes %2Fapp%2Fbrowse)
  3. User logs in
  4. Instead of being forwarded to the default path /app/me, user is sent to the "goto URL", i.e. /app/browse
katamatata commented 6 months ago

@astkhikatredi, I want to mention these points:

If you think it can be a problem, let's discuss & create a ticket.

ericbolikowski commented 6 months ago

Deployed!