Skip to content

Show "This page requires authentication" page for protected routes if unauthenticated #4315

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

Merged
merged 1 commit into from
Dec 23, 2021

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Dec 22, 2021

Bildschirmfoto 2021-12-22 um 20 02 10

Resolves #3687

Note that this is based on #4314 and includes those commits to avoid merge conflicts.

@Turbo87 Turbo87 added C-enhancement ✨ Category: Adding new behavior or a change to the way an existing feature works A-frontend 🐹 labels Dec 22, 2021
@Turbo87 Turbo87 force-pushed the please-authenticate branch from fdcb505 to 1d22d15 Compare December 22, 2021 19:20
@Turbo87 Turbo87 marked this pull request as ready for review December 22, 2021 19:20
@Turbo87
Copy link
Member Author

Turbo87 commented Dec 23, 2021

@bors r+

@bors
Copy link
Contributor

bors commented Dec 23, 2021

📌 Commit 1d22d15 has been approved by Turbo87

@bors
Copy link
Contributor

bors commented Dec 23, 2021

⌛ Testing commit 1d22d15 with merge cdab896...

@bors
Copy link
Contributor

bors commented Dec 23, 2021

☀️ Test successful - checks-actions
Approved by: Turbo87
Pushing cdab896 to master...

@bors bors merged commit cdab896 into rust-lang:master Dec 23, 2021
@Turbo87 Turbo87 deleted the please-authenticate branch December 24, 2021 00:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-frontend 🐹 C-enhancement ✨ Category: Adding new behavior or a change to the way an existing feature works
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Login error covers login link
2 participants