Project

General

Profile

Feature #27644

Check a user's permission before allowing them to access react pages

Added by Jeremy Lenz 8 months ago. Updated 6 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
JavaScript stack
Target version:
-
Difficulty:
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Currently we allow users to visit pages they don't have access to. If a user doesn't have permission to view a page we shouldn't allow them to view it.

To be used in concert with Katello issue #27450


Related issues

Copied from Katello - Feature #27450: Check a user's permission before allowing them to access react pages Closed

Associated revisions

Revision 8e111f83 (diff)
Added by Jeremy Lenz 7 months ago

Fixes #27644: Add PermissionDenied component

add missing_permissions to rabl view
Implement PR review suggestions
Add index.js
Add tests & snaps
Add ForemanLink component
Add storybook page (couldn't test due to 27531 not merged)
update snapshot
move storybook to Components level
exclude static storybook from git
switch back to just using <a>
update snaps
pass strings literal to translate
Update test to use testComponentSnapshotsWithFixtures
remove old test
remove .gitignore line per walden comment

History

#1 Updated by Jeremy Lenz 8 months ago

  • Copied from Feature #27450: Check a user's permission before allowing them to access react pages added

#2 Updated by The Foreman Bot 8 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/6979 added

#3 Updated by Jeremy Lenz 8 months ago

  • Pull request deleted (https://github.com/theforeman/foreman/pull/6979)

#4 Updated by Walden Raines 8 months ago

  • Pull request https://github.com/theforeman/foreman/pull/6979 added
  • Pull request deleted (https://github.com/Katello/katello/pull/8262)

#5 Updated by The Foreman Bot 7 months ago

  • Fixed in Releases 1.24.0 added

#6 Updated by Jeremy Lenz 7 months ago

  • Status changed from Ready For Testing to Closed

#7 Updated by Amit Upadhye 6 months ago

  • Category set to JavaScript stack

Also available in: Atom PDF