Skip to content
Snippets Groups Projects

Axe session expired functionality: redir to /login

Open Axe session expired functionality: redir to /login
debeule-master-patch-31879 into master
Open Matthieu De Beule requested to merge debeule-master-patch-31879 into master

Might be nice to have a message shown on /login page, but it's annoying to have to click through to the login page each time: why not redirect there immediately?

Merge request reports

Pipeline #2198 passed

Stage:
    Stage:
      Stage:

        Pipeline passed for 2b9aec83 on debeule-master-patch-31879

        Checking approval status
        Ready to merge by members who can write to the target branch.

        Merge details

        • The source branch is 38 commits behind the target branch.
        • 0 commits and 1 merge commit will be added to .
        • Source branch will not be deleted.

        Activity

        Filter activity
        • Approvals
        • Assignees & reviewers
        • Comments (from bots)
        • Comments (from users)
        • Commits & branches
        • Edits
        • Labels
        • Lock status
        • Mentions
        • Merge request status
        • Tracking
        • A
          Antoine Fontaine @afontain ·
          Developer

          I've made the logout timer longer. I kinda like having the logout dialog, as it explains why you are not on the expected page. Having an automated redirection would be nice though.

          Do you still think this MR is useful?

        • D
          Matthieu De Beule @debeule ·
          Author Developer

          Could we not go to the login page directly and show a message there?

        • A
          Antoine Fontaine @afontain ·
          Developer

          Ok, so another design could be "redirect to /login?page=/subscriptions" (assuming you were on the subscriptions page), present a message here; and on login, you get redirected there. Would that work for you?

          Edited by Antoine Fontaine
        Please register or sign in to reply