-
Notifications
You must be signed in to change notification settings - Fork 44
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
List all organizations #1345
List all organizations #1345
Conversation
Org List and delete modal, Updated request data org-status-update and org-delete
Codecov Report
@@ Coverage Diff @@
## master #1345 +/- ##
==========================================
+ Coverage 73.75% 73.97% +0.21%
==========================================
Files 171 172 +1
Lines 5179 5271 +92
Branches 1074 1089 +15
==========================================
+ Hits 3820 3899 +79
- Misses 996 1005 +9
- Partials 363 367 +4
Continue to review full report at Codecov.
|
…to list-all-organizations
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please change organization to organizations everywhere.
title: 'Organization', | ||
path: '/organization', | ||
type: 'staffManagement', | ||
roles: ['Manager', 'Admin'], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just give rights for admin only.
src/context/role.ts
Outdated
/** | ||
* Removing organization list menu for non admin users | ||
*/ | ||
if (user.name !== 'NGO Main Account') { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The user will be Saas Admin. Let's discuss and confirm in the call.
…to list-all-organizations
with role manager or admin will not able to edit glific-admin
…to list-all-organizations
closes #1323