Notifications "commentaires" souci ergo mineur

cyrilmorin commented 3 weeks ago

Hello,

Je viens d'identifier une souci mineur sur le 2nd datasets sur la notif réutilisateur sur les commentaires. En effet dans le corp du mail la 2nde ressource est sur un niveau de puce différent.

AntoineAugusti commented 2 weeks ago

Merci.

Peux-tu partager le code HTML de cet e-mail ?

In Microsoft Outlook, double-click to open an email. You'll see an “Actions” menu under the “Message” tab. Click on that menu and select the “Other Actions,” then click on “View Source” to see the HTML code.

Ou quelque chose de similaire.

cyrilmorin commented 2 weeks ago

Je t'ai rajouté à l'email de Front !

Bonjour,

Des discussions ont eu lieu sur certains jeux de données que vous suivez. Vous pouvez prendre connaissance de ces échanges.

Réseau interurbain BreizhGo Car
- Réseau interurbain BreizhGo Bateaux
L’équipe transport.data.gouv.fr

AntoineAugusti commented 2 weeks ago

E-mail concerné dans Front (on peut voir le HTML source là-bas)

<p>
Bonjour,</p>
<p>
Des discussions ont eu lieu sur certains jeux de donn=C3=A9es que vous suiv=
ez. Vous pouvez prendre connaissance de ces =C3=A9changes.</p>
<p>
</p>
<ul>
  <li>
<a href=3D"https://mailjet-link-tracking.transport.data.gouv.fr/lnk/EAAABaV=
JM64AAAAAAAAAASqWiDsAAAAARUgAAAAAABGtEwBmaAQSvteEmusPSo27xYz8PFv1MgARuyQ/1/=
6rsBOjnjBPKDHrwwCO-bOw/aHR0cHM6Ly90cmFuc3BvcnQuZGF0YS5nb3V2LmZyL2RhdGFzZXRz=
L2JyZWl6aGdvLWNhciNkYXRhc2V0LWRpc2N1c3Npb25z#dataset-discussions">R=C3=A9se=
au interurbain BreizhGo Car</a>
</p>
<ul>
  <li>
<a href=3D"https://mailjet-link-tracking.transport.data.gouv.fr/lnk/EAAABaV=
JM64AAAAAAAAAASqWiDsAAAAARUgAAAAAABGtEwBmaAQSvteEmusPSo27xYz8PFv1MgARuyQ/2/=
fpMg2Y5nJyhZnuoY6HXKLw/aHR0cHM6Ly90cmFuc3BvcnQuZGF0YS5nb3V2LmZyL2RhdGFzZXRz=
L2JyZWl6aGdvLWJhdGVhdXgjZGF0YXNldC1kaXNjdXNzaW9ucw#dataset-discussions">R=
=C3=A9seau interurbain BreizhGo Bateaux</a>
  </li>
</ul>
<p>
L=E2=80=99=C3=A9quipe transport.data.gouv.fr</p>

Le HTML de la liste des liens n'est pas correct.

AntoineAugusti commented 2 weeks ago

@ptitfred Serais-tu intéressé par ce ticket ?

Visiblement le Markdown to HTML sur ce template/ligne https://github.com/etalab/transport-site/blob/9ecf35543bd1aa836981d4dd2d2630c8126081a4/apps/transport/lib/transport_web/templates/email/new_comments_reuser.html.md?plain=1#L6 se déroule mal. Voir si en changeant un peu le Markdown ça passe, ou mettre directement des balises HTML dans le template ?

Voir pour adapter le test aussi, montrer qu'on ne met pas plusieurs <ul></ul>.

Je t'ai déjà suggéré de travailler sur plusieurs tickets 🐛 dernièrement. Dis-moi si tu veux que je m'en charge.

ptitfred commented 2 weeks ago

J'avais jeté un œil rapide ce matin des fois que ça me saute aux yeux mais pas trouvé. Je vais étendre les tests et fixer le template.

AntoineAugusti commented 2 weeks ago

Ça peut être un bug de phoenix_markdown https://github.com/etalab/transport-site/blob/9ecf35543bd1aa836981d4dd2d2630c8126081a4/apps/transport/mix.exs#L66 ou de l'usage qu'on en fait.

Me semble que j'avais déjà eu des problèmes dans le passé avec des listes dans les e-mails en Markdown, que j'étais passé en HTML ensuite.

Là en regardant le template Markdown le markup me semble bon et ne devrait pas produire ce résultat.

etalab / transport-site

Notifications "commentaires" souci ergo mineur #3989