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

Add permission on SR details page #817

Merged
merged 4 commits into from
Aug 1, 2024
Merged

Conversation

AndrewThien
Copy link
Contributor

@AndrewThien AndrewThien commented Aug 1, 2024

Changes

This PR added a permission check on the ScanReport details page. Also, it possibly made the Forbidden component nicer.

Related #797

Checks

Important: please complete these before merging.

  • Run migrations, if any.
  • Update changelog.md, including migration instructions if any.
  • Run unit tests.

Copy link

github-actions bot commented Aug 1, 2024

Tests Skipped Failures Errors Time
25 1 💤 1 ❌ 0 🔥 4.149s ⏱️

@AndrewThien AndrewThien merged commit 61cd998 into master Aug 1, 2024
14 checks passed
@AndrewThien AndrewThien deleted the fix/add-permission-SR-details branch August 1, 2024 14:41
@AndrewThien AndrewThien changed the title Add permission on layout of SR details page Add permission on SR details page Aug 1, 2024
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.

3 participants