georchestra / helm-georchestra

geOrchestra helm chart
3 stars 6 forks source link

use as much webapps shortcut as possible #73

Closed edevosc2c closed 9 months ago

edevosc2c commented 1 year ago

Everywhere where it's possible, use "webapps" shortcut.

Some places might still have possible refactoring but it would need more thoughts into it.

edevosc2c commented 1 year ago

Oops I think I forgot to double-check, some webapps shortcut were not assigned correctly.

jeanpommier commented 1 year ago

Looks good to me. Did you deploy it for further checks ?

edevosc2c commented 1 year ago

Did you deploy it for further checks ?

Not yet I'll try. At least the github actions checks passed :).

jeanpommier commented 1 year ago

Yes. and helm doesn't complain. :o)

edevosc2c commented 9 months ago

Validated by deploying georchestra helm chart in test cluster.