Change admin redirection process #27

Closed
opened 5 months ago by neil · 1 comments
neil commented 5 months ago
Owner

The current system has been built to force people to use the same admin link. If someone tries to create a new admin link or uses an admin link AND they actually have another sncf session in cookies, it will load the previous session instead of loading a new one.

The stored session must be discarded in favor of the new admin link session.

It was intended to be a feature, but UX tests proved there's something wrong.

Process

  • if user is logged in when creating form link
    • delete nc cookies
    • delete sncf cookies
    • create a new form link as requested instead of redirecting to /apps/forms
  • if user is logged in when browsing /admin/
    AND if token is not the same
    • delete nc cookies
    • delete sncf cookies and override with the newly created session (from the admin link)
The current system has been built to force people to use the same admin link. If someone tries to create a new admin link or uses an admin link AND they actually have another sncf session in cookies, it will load the previous session instead of loading a new one. The stored session must be discarded in favor of the new admin link session. It was intended to be a feature, but UX tests proved there's something wrong. ### Process - if user is logged in when creating form link - delete nc cookies - delete sncf cookies - create a new form link as requested instead of redirecting to /apps/forms - if user is logged in when browsing /admin/ AND if token is not the same - delete nc cookies - delete sncf cookies and override with the newly created session (from the admin link)
neil added the
priority:high
bug
labels 5 months ago
Poster
Owner

Actually there's no need to delete the cookies, they will just get overwritten. I just need to remove the is_logged_in check.
Currently testing, I'll be able to confirm that soon.

Actually there's no need to delete the cookies, they will just get overwritten. I just need to remove the is_logged_in check. Currently testing, I'll be able to confirm that soon.
neil closed this issue 5 months ago
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.