-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Hide license type in admin dashboard when not applicable #9767
Comments
It's my understanding that there will always be a license, it's just that it may not be a license that the user has uploaded. It may be the "default license" (<= 10 users) or the "fallback license" (> 10 users) If you're on the default license, it'll look similar to as it does in the screenshot. If it's the fallback, it'll only show the admin dashboard as an available feature and also show unlimited users. |
@mrsimonemms From what I understand, this is not the case for Gitpod.io, no? Should we at least catch these edge cases and show something like Uknown License? I remember seeing this being added in https://github.com/gitpod-io/gitpod/pull/9343/files#diff-e1d7e44897170d01d2c3ace2ca628317d5ff39129f1b945b29381cad48a50670R128. Cc @nandajavarma |
Just to add to the discussion, in the case of |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Problem to solve
In /admin/license there are some edge cases where license type is not applicable.
Cc @nandajavarma @mrsimonemms because #9763
Proposal
When the license information is not applicable, let's hide the key-value element altogether.
Since we're here, we could also:
🅱️ include some left margin ((being addressed in Change icon spacing in license page #9823)ml-4
) on the license status icon for better visual balance.The text was updated successfully, but these errors were encountered: