Skip to content

Conversation

@prakashsvmx
Copy link
Member

@prakashsvmx prakashsvmx commented Aug 14, 2023

show service account status and expiry in ui
Fixes #2848

How does it look?
image

image

Note: There is no UI to update expiry, name ,description and status . for now it is part of mc commands

@prakashsvmx prakashsvmx self-assigned this Aug 14, 2023
@prakashsvmx prakashsvmx marked this pull request as draft August 14, 2023 14:11
@prakashsvmx prakashsvmx force-pushed the sa-ui-improvements branch 4 times, most recently from 6adf268 to 6c2d9c4 Compare August 16, 2023 08:04
@prakashsvmx prakashsvmx marked this pull request as ready for review August 16, 2023 08:55
Copy link
Collaborator

@cesnietor cesnietor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested disabling svcacct and status is being reflected.

Copy link
Collaborator

@bexsoft bexsoft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, string[] change in PolicyDetails.tsx is fine

@bexsoft bexsoft merged commit e7fb205 into minio:master Aug 16, 2023
@djwfyi djwfyi mentioned this pull request Sep 5, 2023
1 task
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.

Service Account doesn't show status

5 participants