helm-dashboard icon indicating copy to clipboard operation
helm-dashboard copied to clipboard

Query helm chart source for icon and description

Open undera opened this issue 2 years ago • 5 comments

In the list of installed charts, attempt to improve display by issuing additional call to query for icon and description. This would require introducing a new API call, since we don't want to delay the display of the list. This needs to match release to repository, which we already able to do. Then, helm show chart komodorio/k8s-watcher to get the icon URL and description. Make sure the UI is not "jumping" because of variable description length.

undera avatar Oct 28 '22 17:10 undera

I would like to pick this up

harshit-mehtaa avatar Nov 02 '22 14:11 harshit-mehtaa

Go for it

undera avatar Nov 02 '22 14:11 undera

helm show chart gives response back in yaml format is there a way to fetch from yaml response or convert it to json?

harshit-mehtaa avatar Nov 02 '22 14:11 harshit-mehtaa

or is better to do it on go?

harshit-mehtaa avatar Nov 02 '22 14:11 harshit-mehtaa

Parsing the YAML is not too big of a problem. We already work with it like here: https://github.com/komodorio/helm-dashboard/blob/c0a1d31c8d717db1bf5d42ab84275aca3f988ade/pkg/dashboard/subproc/data.go#L269

In case of helm show ... it can be even simpler, since it's a single-document expected in output. https://github.com/helm/helm/blob/main/pkg/chart/metadata.go#L47 should be the type into which it is deserialized, or something around it.

The data layer should do the responsibility of parsing this into objects.

undera avatar Nov 02 '22 14:11 undera