loft-sh / loft

Namespace & Virtual Cluster Manager for Kubernetes - Lightweight Virtual Clusters, Self-Service Provisioning for Engineers and 70% Cost Savings with Sleep Mode
https://loft.sh/docs/introduction
Other
737 stars 65 forks source link

Improve UX for Helm apps #162

Closed kostis-codefresh closed 2 years ago

kostis-codefresh commented 3 years ago

It is not immediately clear that the warning message about the ingress only refers to the cluster subdomain feature.

I don't understand why the Helm apps are "disabled"/grey color

I think the warning should clarify its scope. Now it looks like it refers to both UI segments

FabianKramm commented 3 years ago

@kostis-codefresh thanks for creating this issue! Yes we can separate this more to make it clearer.

kostis-codefresh commented 3 years ago

You can simply change the warning message to "To use the cluster subdomain feature you need to have an ingress controller in your cluster"

The confusing part is that you mention nginx and say "see above" which of course points to the Helm applications and the nginx chart directly above

carlmontanari commented 2 years ago

I think that at least w/ 2.3.2 this is looking saner/not confusing, so I think I am going to close this one out! Feel free to re-open if you are still seeing something confusing/wrong!