Add codimd to Meshapps umbrella chart

[SamuAlfageme]

This also fixes an issue with the default collabora image value as well as some wrong terms in the README.

Closes sciencemesh/sciencemesh#74

[Daniel-WWU-IT]

There are a few more environment variables we've set in our test deployment; not sure if they're really needed:

• CMD_SAVE_WEBHOOK: https://sciencemesh-test.uni-muenster.de/wbridge/save
• CMD_APPLIANCE_MODE: true
• CMD_URL_PATH: codimd

[SamuAlfageme]

@Daniel-WWU-IT thanks a lot for your review ❤️

The CMD_APPLIANCE_MODE set to true is already present on the values that are part of this PR:

charts/meshapps/values.yaml

Line 23 in 1bf6f93

CMD_APPLIANCE_MODE: true

And indeed, the URL_PATH and SAVE_WEBHOOK are considered site-specific values and mentioned on the companion doc PR

[glpatcern]

And indeed, the URL_PATH and SAVE_WEBHOOK are considered site-specific values and mentioned on the companion doc PR

[Reviewing some backlog] I agree with you Samuel, nevertheless any chance to "parameterize" such variables? In particular the CMD_SAVE_WEBHOOK must always look like https://<ingress>/<wopibridge_path>/save.

[SamuAlfageme]

@glpatcern I agree, making the config as lean as possible (as in no. of variables to be passed to deploy the chart) and avoiding duplicated variables/content is always a design goal I try to apply. For instance, that is how we generate the WOPI Server/Bridge External URLs currently.

The issue to generate the CMD_SAVE_WEBHOOK content lies in how the two charts are decoupled: there is no way to access the IOP-specific variables (e.g. the WOPI Bridge ingress hostname and path) from a different chart (MESHApps, in this case).

This could be done if we ever consider creating a meta-umbrella chart that agglutinates these two by using Helm's global chart values.