Skip to content

Fixing permission for admin wizard#6061

Merged
PeerRich merged 7 commits intomainfrom
fix/admin-wizard-permission
Dec 19, 2022
Merged

Fixing permission for admin wizard#6061
PeerRich merged 7 commits intomainfrom
fix/admin-wizard-permission

Conversation

@leog
Copy link
Copy Markdown
Contributor

@leog leog commented Dec 16, 2022

What does this PR do?

Adding a redirection to 400 error when logged in user's role is not ADMIN.

Fixes #6060

Environment: Staging(main branch) / Production

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How should this be tested?

  1. Log in as a regular user and go to /auth/setup and you will get redirected to the 400 error

@leog leog requested a review from a team December 16, 2022 15:50
@vercel
Copy link
Copy Markdown

vercel Bot commented Dec 16, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cal ✅ Ready (Inspect) Visit Preview Dec 19, 2022 at 2:47PM (UTC)

Comment thread apps/web/pages/auth/setup/index.tsx Outdated
Comment thread apps/web/pages/auth/setup/index.tsx Outdated
Comment thread apps/web/pages/auth/setup/index.tsx Outdated
@PeerRich PeerRich enabled auto-merge (squash) December 19, 2022 14:39
Copy link
Copy Markdown
Member

@PeerRich PeerRich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@PeerRich PeerRich merged commit 11a0796 into main Dec 19, 2022
@PeerRich PeerRich deleted the fix/admin-wizard-permission branch December 19, 2022 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CAL-606] Admin Wizard visible for non-admin users

5 participants