Skip to content

Show appropriate message when unauthorized user tried to access the a… - #571

Merged
davorrunje merged 3 commits into
mainfrom
568-auto-logout-unauthorized-users-and-clear-ui-session-in-firebase-auth
Nov 13, 2024
Merged

Show appropriate message when unauthorized user tried to access the a…#571
davorrunje merged 3 commits into
mainfrom
568-auto-logout-unauthorized-users-and-clear-ui-session-in-firebase-auth

Conversation

@harishmohanraj

@harishmohanraj harishmohanraj commented Nov 13, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor

Fixes #568

@harishmohanraj harishmohanraj linked an issue Nov 13, 2024 that may be closed by this pull request
Display Access Denied Message for Unauthorized Users #568
Closed
@codecov

codecov Bot commented Nov 13, 2024 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 27.27273% with 8 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
fastagency/ui/mesop/auth/firebase/firebase_auth.py 20.00% 8 Missing ⚠️
Files with missing lines Coverage Δ
fastagency/ui/mesop/data_model.py 100.00% <100.00%> (ø)
fastagency/ui/mesop/auth/firebase/firebase_auth.py 48.88% <20.00%> (-2.40%) ⬇️

... and 2 files with indirect coverage changes

@davorrunje
davorrunje enabled auto-merge November 13, 2024 13:16
@harishmohanraj
Merge remote-tracking branch 'origin/main' into 568-auto-logout-unaut…
3716a71 
…horized-users-and-clear-ui-session-in-firebase-auth
@davorrunje
davorrunje added this pull request to the merge queue Nov 13, 2024
Merged via the queue into main with commit 432f5a0 Nov 13, 2024
@davorrunje
davorrunje deleted the 568-auto-logout-unauthorized-users-and-clear-ui-session-in-firebase-auth branch November 13, 2024 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@davorrunje davorrunje davorrunje approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Display Access Denied Message for Unauthorized Users

2 participants

@harishmohanraj @davorrunje