Enable support for custom HTML in riot-web homepage

[anadahz]

• Add default template file for homepage HTML
• Add default riot-web config options for homepage

[spantaleev]

Calling the variable something like matrix_riot_web_homepage_template (instead of matrix_riot_web_config) is probably better.

I also wonder whether using and maintaining our own copy of riot-web's home.html is something we want to take on. It seems like something that may easily get out of date.

On the other hand it's nice we can point it to our own "Riot Bot" (or rather, not show that at all).
We may also be able to inject an additional item in the "Running Matrix services" list, which points to the support channel for this playbook (#matrix-docker-ansible-deploy:devture.com).

So maybe it's worth going to the trouble and doing it..

[anadahz]

Calling the variable something like matrix_riot_web_homepage_template (instead of matrix_riot_web_config) is probably better.

Indeed it makes more sense to use this variable instead, I changed the variables accordingly.

I also wonder whether using and maintaining our own copy of riot-web's home.html is something we want to take on. It seems like something that may easily get out of date.

According to the commits history of home.html it seems that is not getting updated so often.

On the other hand it's nice we can point it to our own "Riot Bot" (or rather, not show that at all).

This is being handled by the variable matrix_riot_web_welcome_user_id if it's undefined the bot section will not be displayed.

We may also be able to inject an additional item in the "Running Matrix services" list, which points to the support channel for this playbook (#matrix-docker-ansible-deploy:devture.com).

Good point added the support channel of this playbook.

[spantaleev]

I've found a few more things that should be changed before we can merge this.

[spantaleev]

Thanks for your work on this!

I've submitted a few minor things to be tested/corrected and let's merge this!

[anadahz]

Ready for review.

[spantaleev]

I've merged this and it seems to work!

I've only had to remove the _t() (translation) call for our custom button.
Otherwise the "Support for Matrix Docker Ansible role" label was being prefixed by some other "missing translation" label. Maybe this only happened when another locale was used (besides the default "en"), but..

Thank you for your work on this! 👍

[anadahz]

I've only had to remove the _t() (translation) call for our custom button.

Good catch, thanks!