Skip to content
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

Wording: Make "Logout-Message" friendlier #7744

Closed
tbsbdr opened this issue Oct 5, 2022 · 5 comments · Fixed by #7747
Closed

Wording: Make "Logout-Message" friendlier #7744

tbsbdr opened this issue Oct 5, 2022 · 5 comments · Fixed by #7747
Assignees
Labels
Category:Enhancement Add new functionality Priority:p3-medium Normal priority

Comments

@tbsbdr
Copy link

tbsbdr commented Oct 5, 2022

Problem: The current logout message leaves me with a slight feeling of failure which leads to an unpleasant ux

Current

Screenshot 000708

Expected

proposal for a friendlier message:

  • Logged out because of inactivity
  • You were automatically logged out after 30 minutes of inactivity for security reasons.
  • Log in again

image

Discarded solution

image

@tbsbdr tbsbdr added Priority:p3-medium Normal priority Category:Enhancement Add new functionality labels Oct 5, 2022
@pascalwengerter
Copy link
Contributor

Feel free to assign this to me 🙋‍♂️

@kulmann
Copy link
Contributor

kulmann commented Oct 6, 2022

Not sure if we can print the "inactivity" in all relevant use cases. Also I'm pretty sure that there are other reasons for being logged out as well. If you don't have a refresh token (i.e. no offline_access in the requested scopes) and no internet connection at the time when the access token renewal is supposed to happen your access token just expires and we can't issue a new one without a roundtrip to the IdP.

Fully agree that the wording needs improvement though.

Awesome that you want to take care @pascalwengerter thank you! ❤️

@tbsbdr
Copy link
Author

tbsbdr commented Oct 6, 2022

jep @kulmann had the same concerns

How about just giving "security" as a reason for the logout?

@kulmann
Copy link
Contributor

kulmann commented Oct 6, 2022

Yes, fine by me as well as a first iteration 👍 I guess that users will still be confused about that, but it's way better than before.

@tbsbdr
Copy link
Author

tbsbdr commented Oct 6, 2022

updated initial post.

pascalwengerter added a commit to pascalwengerter/web that referenced this issue Oct 6, 2022
pascalwengerter added a commit to pascalwengerter/web that referenced this issue Oct 7, 2022
pascalwengerter added a commit to pascalwengerter/web that referenced this issue Oct 7, 2022
JammingBen added a commit that referenced this issue Oct 7, 2022
@kulmann kulmann moved this to Done in Web Support Board Jun 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category:Enhancement Add new functionality Priority:p3-medium Normal priority
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants