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: add manager_user and list_user permissions to display user manag… #281

Merged
merged 1 commit into from
Jan 22, 2022

Conversation

tranquanghuy0801
Copy link
Member

Description

Update the User Manager display permissions

Issue(s) addressed

What kind of change(s) does this PR introduce?

  • Enhancement
  • Bug fix
  • Refactor

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Issue

What is the current behavior?

Only show User Manager on left side panel if the user has SUPER_USER policy

What is the new behavior?

show User Manager on left side panel if the user has SUPER_USER or MANAGER_USER or LIST_USER policy

Breaking change

Does this PR introduce a breaking change?

No

Other useful information

@tranquanghuy0801 tranquanghuy0801 requested a review from a team as a code owner January 10, 2022 14:39
@nmcharlton nmcharlton merged commit be8ff32 into Greenstand:master Jan 22, 2022
mitia-anah pushed a commit to mitia-anah/treetracker-admin-client that referenced this pull request Feb 1, 2022
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.

User Manager only shown if user has super_permission policy
2 participants