[Staging] Notifications header about go-api

[tovari]

I still see the problem at the Weekly digest emails on prod and staging.

[szabozoltan69]

Could not reproduce locally. It means that some kind of env variable issue exists.

Locally the notification is generated nicely:

[szabozoltan69]

These '|quote' transformations are new features in https://github.com/IFRCGo/go-api/blob/terraform/deploy/helm/ifrcgo-helm/templates/config/configmap.yaml#L18-L19 – they may cause new "effects" in notification templates as well.
Do you have any quess how could we use them appropriately in e.g. https://github.com/IFRCGo/go-api/blob/terraform/notifications/templates/design/head1.html#L417 (record.resource_uri) without quotation marks? @batpad @geohacker

[szabozoltan69]

Root cause: settings.FRONTEND_URL contains empty string.
If I fix it, result is ok:

(Probably another entity is also running, where the manual change could not be seen; that other notification remained bad.)
Obsolate:
https://stackoverflow.com/questions/36954472/removing-quotes-in-a-django-template could help. But maybe it's easier to strip the variable in the moment of giving it to the renderer, in https://github.com/IFRCGo/go-api/blob/develop/api/management/commands/index_and_notify.py#L160 and similar in that script.

[szabozoltan69]

With giving FRONTEND_URL issue is solved.

[tovari]

Somehow the icon is still missing on all Weekly digest emails:

[szabozoltan69]

Let me check it...