Restrict Access to Asset Configurations for Specific Admin Roles #6309
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
WHAT
🤖 Generated by Copilot at 0078dca
This pull request improves the authorization logic for managing different asset types in the
AssetManage
component. It uses theuseAuthUser
hook to check the user roles and restrict the access to the configuration forms forHL7Monitor
andONVIFCamera
components to admin users only.Proposed Changes
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
HOW
🤖 Generated by Copilot at 0078dca
useAuthUser
hook inHL7Monitor.tsx
andONVIFCamera.tsx
to get the current user's type and role (link, link)authUser
variable using the hook in both components (link, link)authUser
, to only show the forms to users withDistrictAdmin
orStateAdmin
roles (link, link)authorizeFor
prop from theAssetManage
component inAssetManage.tsx
, since the authorization logic is handled by the child components (link, link)