-
Notifications
You must be signed in to change notification settings - Fork 261
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
New tenants list #1160
New tenants list #1160
Conversation
dvaldivia
commented
Oct 28, 2021
•
edited
edited
be5c137
to
7036248
Compare
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.
LGTM, Just a minor change in the license disclaimer
portal-ui/src/screens/Console/Tenants/TenantDetails/TenantDetails.tsx
Outdated
Show resolved
Hide resolved
portal-ui/src/screens/Console/Tenants/ListTenants/ListTenants.tsx
Outdated
Show resolved
Hide resolved
portal-ui/src/screens/Console/Tenants/ListTenants/ListTenants.tsx
Outdated
Show resolved
Hide resolved
portal-ui/src/screens/Console/Tenants/ListTenants/ListTenants.tsx
Outdated
Show resolved
Hide resolved
portal-ui/src/screens/Console/Tenants/ListTenants/TenantListItem.tsx
Outdated
Show resolved
Hide resolved
portal-ui/src/screens/Console/Tenants/TenantDetails/TenantSummary.tsx
Outdated
Show resolved
Hide resolved
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.
LGTM
@prakashsvmx good point, we have the same problem in other screens and we need to fix it, I'll address that for tenants on a separate tenant |
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.
Tested. Looks good to me.