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

Don't display the login page for logged in users #15934

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hellcp-work
Copy link
Contributor

Avoids displaying the login page if you are already logged in. Meant to fix #6365

@github-actions github-actions bot added the Frontend Things related to the OBS RoR app label Apr 4, 2024
@saraycp saraycp added the review-app Apply this label if you want a review app started label Apr 4, 2024
@obs-bot
Copy link
Collaborator

obs-bot commented Apr 4, 2024

Review app will appear here: http://obs-reviewlab.opensuse.org/hellcp-work-login-page

@saraycp
Copy link
Contributor

saraycp commented Apr 8, 2024

@hellcp-work you have a few feature specs failing...

@hellcp-work
Copy link
Contributor Author

@hellcp-work you have a few feature specs failing...

Indeed, feature specs are trying to log in without ever logging out, which is not supported with this change, I will see how difficult it would be do make sure this doesn't happen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Frontend Things related to the OBS RoR app review-app Apply this label if you want a review app started
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Logging in via [BS instance]/session/new is weird (UX)
3 participants