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

Fix/1474 parent read users not seeing summary panel #6299

Merged
merged 3 commits into from
Jul 24, 2024

Conversation

duncanc19
Copy link
Collaborator

Work done

  • Removed a check for canViewListOfWorkers permission on summary panel in parent home tab so that the panel is rendered for parent read only users
  • Added a check for canViewListOfWorkers to display the View staff records link in the workplace tab so that users without permission to view the staff tab don't see it

Tests

Does this PR include tests for the changes introduced?

  • Yes
  • No, I found it difficult to test
  • No, they are not required for this change

@duncanc19 duncanc19 merged commit 2cf482f into main Jul 24, 2024
5 checks passed
@duncanc19 duncanc19 deleted the fix/1474-parent-read-users-not-seeing-summary-panel branch July 24, 2024 13:08
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.

2 participants