Skip to content
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

Fehlerhafte GID Link im Mail #54

Closed
larsUE opened this issue Apr 20, 2023 · 12 comments
Closed

Fehlerhafte GID Link im Mail #54

larsUE opened this issue Apr 20, 2023 · 12 comments
Assignees
Labels
bug Something isn't working ready for PROD Ticket ready for deployment on PROD

Comments

@larsUE
Copy link
Collaborator

larsUE commented Apr 20, 2023

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

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

@larsUE larsUE created this issue from a note in cokreation.winterthur.ch (Ready) Apr 20, 2023
@bsantschi
Copy link
Collaborator

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

@bsantschi bsantschi moved this from Ready to Testable on INT in cokreation.winterthur.ch Apr 21, 2023
@larsUE
Copy link
Collaborator Author

larsUE commented Apr 21, 2023

Funktioniert auf INT wieder, danke!

@larsUE larsUE closed this as completed Apr 21, 2023
@larsUE larsUE moved this from Testable on INT to Ready in cokreation.winterthur.ch Apr 24, 2023
@larsUE larsUE reopened this Apr 24, 2023
@larsUE larsUE assigned Kagemaru and unassigned larsUE Apr 24, 2023
@larsUE
Copy link
Collaborator Author

larsUE commented Apr 24, 2023

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 larsUE added the bug Something isn't working label Apr 24, 2023
@larsUE
Copy link
Collaborator Author

larsUE commented Apr 28, 2023

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
Copy link
Collaborator

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
Copy link
Collaborator Author

larsUE commented May 11, 2023

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

@larsUE
Copy link
Collaborator Author

larsUE commented May 30, 2023

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

  • Letzte Aktivität auf der Startseite, Beispiel hier

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

  • E-Mail, dass ein neuer Vorschlag in einem Projekt hinzugefügt wurde, dass ich folge
    Bildschirm­foto 2023-05-30 um 10 53 33
    (Beispiel aus Zürich)

Falls dies einfache Fixes sind, dann bitte auch noch die Moderationsansicht (siehe Bild oben flicken). Ansonsten lassen wir es bleiben.

@bsantschi bsantschi assigned Kagemaru and unassigned carlobeltrame Jun 20, 2023
@Kagemaru
Copy link
Collaborator

Kagemaru commented Jun 26, 2023

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
Copy link
Collaborator

@Kagemaru dasselbe habe ich in #54 (comment) auch schon gesagt. Lars möchte das an den zwei ganz spezifischen Stellen gefixt haben.

@Kagemaru
Copy link
Collaborator

Kagemaru commented Jun 26, 2023

@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
Copy link
Collaborator

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 larsUE added the ready for PROD Ticket ready for deployment on PROD label Jun 30, 2023
@larsUE
Copy link
Collaborator Author

larsUE commented Jul 8, 2023

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

Image

@larsUE larsUE closed this as completed Jul 8, 2023
@larsUE larsUE moved this from Ready to Done in cokreation.winterthur.ch Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ready for PROD Ticket ready for deployment on PROD
Development

No branches or pull requests

4 participants