Skip to content

Conversation

@Alevsk
Copy link
Contributor

@Alevsk Alevsk commented Dec 3, 2021

  • Some menu options were not showing even if the user has access to
    perform the operations (IAM Policies)
  • Deleted unecessary backend endpoints.go logic, instead using
    SecureComponent to validate application routes and Menu options
    rendering
  • All the logic related to routes and permissions is now in the
    permissions.ts file
  • Added SecureComponent to List Users page
  • Separated Menu options and routing logic for AdminConsole and
    OperatorConsole
  • Tools are hidden if user don't have access to them or MinIO is running
    in fs mode (heal, audit log, etc
  • Hide change-password button if user don't have access
  • Hide create user button if user don't have access
  • fixed some bugs when ldap/oidc is enabled

Screen Shot 2021-12-09 at 19 08 13

image
Screen Shot 2021-12-10 at 12 17 48
Screen Shot 2021-12-10 at 13 09 57

@Alevsk Alevsk force-pushed the refactor-console-endpoints branch 3 times, most recently from d905286 to 7345681 Compare December 3, 2021 10:15
@Alevsk Alevsk self-assigned this Dec 3, 2021
@Alevsk Alevsk added the WIP This PR is WIP and cannot be merged yet label Dec 3, 2021
@Alevsk Alevsk changed the title Application routing now uses SecureComponent and hasPermission function [WIP] Application routing now uses SecureComponent and hasPermission function Dec 3, 2021
@kaankabalak kaankabalak self-requested a review December 7, 2021 19:24
@Alevsk Alevsk force-pushed the refactor-console-endpoints branch 2 times, most recently from 89c1ca3 to 128daff Compare December 7, 2021 20:30
@Alevsk Alevsk changed the title [WIP] Application routing now uses SecureComponent and hasPermission function [WIP DON'T MERGE] Application routing now uses SecureComponent and hasPermission function Dec 8, 2021
@Alevsk Alevsk force-pushed the refactor-console-endpoints branch 3 times, most recently from dd48932 to a1f14c0 Compare December 10, 2021 03:06
@Alevsk Alevsk changed the title [WIP DON'T MERGE] Application routing now uses SecureComponent and hasPermission function Application routing now uses SecureComponent and hasPermission function Dec 10, 2021
@Alevsk Alevsk removed the WIP This PR is WIP and cannot be merged yet label Dec 10, 2021
@Alevsk Alevsk force-pushed the refactor-console-endpoints branch 4 times, most recently from 27b131f to c1fbb01 Compare December 10, 2021 21:10
@Alevsk Alevsk added the WIP This PR is WIP and cannot be merged yet label Dec 10, 2021
@Alevsk Alevsk changed the title Application routing now uses SecureComponent and hasPermission function [WIP] Application routing now uses SecureComponent and hasPermission function Dec 10, 2021
- Some menu options were not showing even if the user has access to
  perform the operations (IAM Policies)
- Deleted unecessary backend endpoints.go logic, instead using
  SecureComponent to validate application routes and Menu options
  rendering
- All the logic related to routes and permissions is now in the
  permissions.ts file
- Added SecureComponent to List Users page
- Separated Menu options and routing logic for AdminConsole and
  OperatorConsole
- Tools are hidden if user don't have access to them or MinIO is running
  in fs mode (heal, audit log, etc
- Hide change-password button if user don't have access
- Hide create user button if user don't have access
- fixed some bugs when ldap/oidc is enabled

Signed-off-by: Lenin Alevski <[email protected]>
@Alevsk Alevsk force-pushed the refactor-console-endpoints branch from c1fbb01 to 892c923 Compare December 11, 2021 03:31
@Alevsk Alevsk removed the WIP This PR is WIP and cannot be merged yet label Dec 13, 2021
@Alevsk Alevsk changed the title [WIP] Application routing now uses SecureComponent and hasPermission function Application routing now uses SecureComponent and hasPermission function Dec 13, 2021
@harshavardhana harshavardhana merged commit 3b2c740 into minio:master Dec 14, 2021
@harshavardhana harshavardhana deleted the refactor-console-endpoints branch December 14, 2021 06:37
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.

3 participants