decidim-swiss / decidim.swiss

2 stars 0 forks source link

Fehlerhafte GID Link im Mail #54

Closed larsUE closed 1 year ago

larsUE commented 1 year ago

Hashtags in Mails werden nicht als Link, sondern als GID angezeigt

Bildschirm­foto 2023-04-20 um 16 31 33

bsantschi commented 1 year ago

Hoi Lars, könntest du das bitte auf INT testen?

larsUE commented 1 year ago

Funktioniert auf INT wieder, danke!

larsUE commented 1 year ago

Zu früh gefreut, ich habe nach wie vor fehlerhafte Hashtags/GID-Darstellung in den Mails, z.B. beim melden eines Beitrags:

Bildschirm­foto 2023-04-24 um 12 19 03
larsUE commented 1 year ago

Der Fehler tritt auch in der User-Activity auf: https://cokreation.winterthur.ch/profiles/lars_kaiser/activity

@Kagemaru: Konntest du dir da bereits was anschauen?

carlobeltrame commented 1 year ago

Das sind schlicht Bugs in Decidim. Hashtags in Titeln und Beschreibungen und Kommentaren werden bisher an manchen Orten im UI richtig präsentiert, aber intern wirds in dem GID-Format gespeichert. Standardmässig wird daher überall, wo keine Extra-Logik vorhanden ist, die GID gerendert. Das ist nichts neues, ist z.B. bei den Kommentaren mindestens seit 3 Jahren schon so, soweit ich anhand dem Code verstehe.

Jede Stelle die du hier auflistest müssen wir oder Decidim einzeln fixen. Möchtest du, dass wir damit beginnen?

larsUE commented 1 year ago

Ja, gerne beheben im obersten E-Mail und der letzten Aktivität auf der Startseite (siehe hier: https://cokreation.winterthur.ch)

larsUE commented 1 year ago

@carlobeltrame : Ich habe noch etwas getestet und sehe die Links nur bei den zwei Fällen. Bitte also diese Cases flicken:

Bildschirm­foto 2023-05-30 um 10 51 55

Kagemaru commented 1 year ago

Letzte Aktivität auf der Startseite, Beispiel hier

Dies scheint ein Problem zu sein, wenn Hashtags im Titel vorkommen. So wie ich das gesehen habe, bricht das an vielen Stellen, weil Decidim dies nicht erwartet.

carlobeltrame commented 1 year ago

@Kagemaru dasselbe habe ich in https://github.com/decidim-swiss/decidim.swiss/issues/54#issuecomment-1531563794 auch schon gesagt. Lars möchte das an den zwei ganz spezifischen Stellen gefixt haben.

Kagemaru commented 1 year ago

@carlobeltrame Ja danke, den Thread habe ich gelesen.

Ich meinte damit, dass es bei mir an anderen Stellen übel kaputt ist, wenn man einen Link im Titel hat. Ich weiss nur nicht, wie nachhaltig diese Changes sind. ~Aber auf PROD kann ich es nicht reproduzieren. Daher fixe ich es mal.~

Siehe hier: Image

Edit:

Ich konnte es reproduzieren, mit Änderungsvorschlägen aktiviert:

https://cokreation.winterthur.ch/processes/marktplatz/f/470/proposals/431 Image

Kagemaru commented 1 year ago

Die Fixes sind nun auf INT. Es gibt noch ein seltsames Verhalten mit den Mails, die Follower erhalten. Dort funktioniert es manchmal und manchmal nicht, trotz geleerten Caches.

Laut Lars ist dies vorerst ok so.

larsUE commented 1 year ago

Neu entdeckter Case. Da es ein sehr seltenes Mail ist, bitte keine Zeit investieren.

Image