Skip to content

Dev/Preprod - set "Tchap" as brand name #923

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 27, 2024

Conversation

mcalinghee
Copy link
Contributor

@mcalinghee mcalinghee commented Feb 28, 2024

Contexte

Le css utilisé dans les templates mail/html sont dépendants du « brand » de Tchap Web.
Ces css ne marchent plus en dev et en preprod lors de la génération des mails.

Ce « brand » est défini dans les config.json de Tchap Web :

  • en Prod : brand = Tchap
  • en Preprod : brand = TCHAP PREPROD
  • en Dev : brand = TCHAP DEV

Ticket impacté : https://github.com/tchapgouv/tchap-infra/pull/2483

Objectif

Changer les configurations de Tchap Web afin de fixer l'utilisation des css dans les templates mails pour dev et preprod et d'être iso avec la prod.

@mcalinghee mcalinghee changed the title set "Tchap" as brand name for dev and preprod Dev/Preprod - set "Tchap" as brand name Feb 28, 2024
@odelcroi
Copy link
Member

On utilise cette distinction pour les environnements, n'y aurait-il pas une manière de faire plusieurs templates mail avec une brand differente? Ou alors de parametriser cette valeur selon les environnements dans le template?

@mcalinghee
Copy link
Contributor Author

mcalinghee commented Feb 28, 2024

On utilise cette distinction pour les environnements, n'y aurait-il pas une manière de faire plusieurs templates mail avec une brand differente? Ou alors de parametriser cette valeur selon les environnements dans le template?

Pour info, le "brand" est utilise en dur dans synapse.

  1. Est ce qu'on pourrait utiliser une autre config que le "brand" pour distinguer les environnements?
  2. On pourrait changer les process d'installation cote infra selon les envs mais plus lourd comme changement => et faudrait quand meme changer le brand dans Tchap Web pour retirer les espaces dans le "brand"

@odelcroi
Copy link
Member

Ok
Nous avons d'autres moyen de différencier les environnements lors de l'utilisation de tchap web :

  • auth_header_logo_url : le logo du login
  • le user notice "user_notice" qui s'affiche à la connection

@odelcroi
Copy link
Member

odelcroi commented Feb 28, 2024

Le css utilisé dans les templates mail/html sont dépendants du « brand » de Tchap Web.

Sais tu pointer où cette liaison est faite? J'ai du mal a voir le lien entre la config de Tchap Web et les mails de notifications qui partent de Synapse

@mcalinghee
Copy link
Contributor Author

Le css utilisé dans les templates mail/html sont dépendants du « brand » de Tchap Web.

Sais tu pointer où cette liaison est faite? J'ai du mal a voir le lien entre la config de Tchap Web et les mails de notifications qui partent de Synapse

Concernant le css :

  • par defaut, Synapse se base sur la valeur qu'on lui met en configuration : app_name
  • si on utilise des liens générés dans les emails, Synapse va se baser sur le brand du serveur front (ici c'est Tchap Web)
# Enable sending emails for notification events
# Defining a custom URL for Riot is only needed if email notifications
# should contain links to a self-hosted installation of Riot; when set
# the "app_name" setting is ignored.

@NicolasBuquet
Copy link

Pour info, pour distinguer les versions sur mobile :

  • sur iOS, on parle de target
  • sur Android, on parle de flavor

Si ça peut vous inspirer…

@odelcroi
Copy link
Member

odelcroi commented Mar 5, 2024

@mcalinghee ok, merci c'est très clair.

Je comprend que le pusher est créé depuis le client web, voilà pourquoi, dans le pusher config, on récupère le brand name du client web. Par exemple sur preprod TCHAP PREPROD.

    1. Donc on peut aligner le app_name de synapse sur celui de tchap web, mais cela peut avoir des effets de bord sur synapse
    1. on met le meme brand name sur tous les tchap web

On peut partir sur 2), qu'ne penses-tu @estellecomment ?

@estellecomment
Copy link
Contributor

Des effets de bord sur synapse, ca semble dangereux à vue de nez (mais j'ai pas tout tout tout suivi je vous laisse juger)
Alors que sur web, c'est surtout de la convenience, pour essayer de pas faire d'erreurs d'etourderie.

Donc oui, on peut revenir au meme brand name sur web pour les 3 env, je valide :)

(Ca nous enleve qq petits indices sympa dans l'interface web, m'enfin au pire on peut regarder les requetes serveur pour verifier l'env ou on est. Et on pourrait rajouter d'autres indices si ca manque, genre un log special dans la console ou chepaquoi... Pas un gros probleme.)

@odelcroi odelcroi self-requested a review March 13, 2024 13:54
@mcalinghee mcalinghee merged commit 735ad77 into develop_tchap Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants