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

Falsche Übersetzungen (Caching?) #21

Closed
innosmith opened this issue Sep 14, 2022 · 8 comments
Closed

Falsche Übersetzungen (Caching?) #21

innosmith opened this issue Sep 14, 2022 · 8 comments
Assignees
Labels
ready for PROD Ticket ready for deployment on PROD

Comments

@innosmith
Copy link
Collaborator

innosmith commented Sep 14, 2022

Siehe Sreenshots ist der Email Betreff auf den Tenant deinKlima angepasst, Emails werden jedoch mit dem falschen Subject "Open Community Luzern ..." versendet. Den Cache habe ich mehrfach geleert, Problem bleibt bestehen. Haben wir ein Caching-Problem oder ist diese falsche Übersetzung allenfalls irgendwo noch auf Code-Ebene definiert, weshalb die Übersetzung mit Term Customizer nicht greift?

Getetest auf PROD, da INT nur auf Einladung und somit keine Registrierung zugelassen.

Bildschirmfoto 2022-09-14 um 08 49 37

Bildschirmfoto 2022-09-14 um 08 42 24

@innosmith innosmith created this issue from a note in deinklima.ch (Ready) Sep 14, 2022
@bsantschi
Copy link
Collaborator

bsantschi commented Sep 14, 2022

Möglicherweise dasselbe Problem, das @Kagemaru hier bereits am Anschauen ist: stadtluzern/decidim-ocl#225

@innosmith
Copy link
Collaborator Author

Möglicherweise dasselbe Problem, das @Kagemaru hier bereits am Anschauen ist: stadtluzern/decidim-ocl#225

Sehr wahrscheinlich dasselbe Problem @bsantschi , @Kagemaru , @Naadjjaa . Für deinKlima könnte das zum Launch-"Bremser" werden.

@larsUE
Copy link
Collaborator

larsUE commented Sep 15, 2022

Ist auf gemeinsamer.space auch so, trotz Term Customizer Anpassung.
Ich sehe, dass die Luzern im Sprachfile so erwähnt wird. Mein Vorschlag wäre nun, "Open Community Luzern" (das ist ja nicht mal für Luzern richtig) durch %{organization} zu ersetzen. Das könnte ich machen, danach bräuchten wir einen INT-Release um zu testen.

Was denkt ihr, @Kagemaru @Naadjjaa und @innosmith ?

@innosmith
Copy link
Collaborator Author

Wenn das funktioniert @larsUE , sehr gerne umsetzen.

larsUE added a commit to larsUE/decidim-ocl that referenced this issue Sep 15, 2022
@Kagemaru Kagemaru moved this from Ready to In progress in deinklima.ch Sep 15, 2022
@Kagemaru
Copy link
Collaborator

@bsantschi: Ja, es ist das gleiche Problem

@larsUE: Ich glaube nicht, dass die Organisation dort reingegeben wird, somit ist sie dann auch nicht im String verwendbar

@bsantschi, @Naadjjaa, @innosmith, @larsUE:
Ich denke, ich habe nun den Ursprung gefunden, wo es schiefläuft (leider nicht warum). Ein Fix dafür ist es, die Mail Templates zu überschreiben, die hier verwendet werden.
Beim nächsten INT Release, wird dies testbar sein. Ich update hier, wenn es deployt ist.

@Kagemaru
Copy link
Collaborator

Ich habe unser Problem bei TermCustomizer als Issue erstellt:
mainio/decidim-module-term_customizer#93

@bsantschi
Copy link
Collaborator

bsantschi commented Sep 27, 2022

Bis der Fehler bei TermCustomizer geflickt wird, könnten wir das Problem umgehen, indem wir generische/gemeinsame E-Mailtitel für alle Tenants definieren. Die E-Mail-Inhalte sollten wir pro Tenant unterschiedlich machen können, jedoch nicht im TermCustomizer, sondern direkt im Code. Folgende Übersetzungen sind davon betroffen:

Edit Thomas: Translation keys, siehe unten

@Kagemaru
Copy link
Collaborator

Kagemaru commented Oct 6, 2022

Dies sollte jetzt endlich behoben sein: https://github.com/stadtluzern/decidim-ocl/projects/1#card-85328903

Folgende Keys wurden für dialogluzern spezifisch übersetzt:

--- 
devise: 
  mailer: 
    confirmation_instructions: 
      action: "Bestätige dein Konto"
      greeting: "Willkommen %{recipient}!"
      instruction: "Du kannst deine E-Mail-Adresse über den folgenden Link bestätigen:"
    email_changed: 
      greeting: "Hallo %{recipient}!"
      message: "Wir schreiben dir, um dich darüber zu informieren, dass deine E-Mail-Adresse zu %{email} geändert wurde."
      message_unconfirmed: "Wir schreiben dir, um dich darüber zu informieren, dass deine E-Mail-Adresse zu %{email} geändert wurde."
    invitation_instructions: 
      accept: "Die Einladung annehmen"
      accept_until: "Diese Einladung wird in %{due_date} fällig."
      decline: "Einladung ablehnen"
      hello: "Hallo %{email},"
      ignore: "Wenn du die Einladung nicht annehmen möchtest, ignoriere diese E-Mail.<br /> Dein Konto wird erst erstellt, Wenn du auf den obigen Link zugreifst und deinen Benutzernamen und dein Passwort festlegst."
      invited_you_as_admin: "%{invited_by} hat dich als Admin von %{application} eingeladen. Du kannst es über den folgenden Link akzeptieren."
      invited_you_as_private_user: "%{invited_by} hat dich als private/n Benutzer/in von %{application} eingeladen. Du kannst es über den folgenden Link akzeptieren."
      someone_invited_you: "Jemand hat dich zu %{application} eingeladen. Du kannst die Einladung über den folgenden Link akzeptieren."
      someone_invited_you_as_admin: "Jemand hat dich als Admin von %{application} eingeladen, du kannst die Einladung über den untenstehenden Link akzeptieren."
      someone_invited_you_as_private_user: "Jemand hat dich als private_user von %{application} eingeladen, du kannst die Einladung über den folgenden Link akzeptieren."
    password_change: 
      greeting: "Hallo %{recipient}!"
      message: "Wir kontaktieren dich, um dich darüber zu informieren, dass dein Passwort geändert wurde."
      subject: "Passwort geändert"
    reset_password_instructions: 
      action: "Mein Passwort ändern"
      greeting: "Hallo %{recipient}!"
      instruction: "Jemand hat einen Link angefordert, um dein Passwort zu ändern. Falls du das selbst warst, dann folge diesem Link:"
      instruction_2: "Falls du das nicht selbst warst, kannst du diese E-Mail ignorieren."
      instruction_3: "Dein Passwort ändert sich erst, wenn du auf den obigen Link zugreifst und ein neues Passwort erstellst."
    unlock_instructions: 
      action: "Entsperre mein Konto"
      greeting: "Hallo %{recipient}!"
      instruction: "Klicke auf den folgenden Link, um dein Konto zu entsperren:"
      message: "Dein Konto wurde aufgrund einer übermässigen Anzahl fehlgeschlagener Anmeldeversuche gesperrt."

Falls ihr auch eigene spezifische Übersetzungen wollt, brauche ich alle diese Einträge übersetzt (ganz oder gar nicht).
Ansonsten werden die Standardübersetzungen verwendet. Wenn sich alle einig sind, kann man auch daran noch schrauben.

@Kagemaru Kagemaru closed this as completed Oct 6, 2022
deinklima.ch automation moved this from In progress to Review Oct 6, 2022
@Kagemaru Kagemaru moved this from Review to Testable on INT in deinklima.ch Oct 6, 2022
@innosmith innosmith added the ready for PROD Ticket ready for deployment on PROD label Oct 8, 2022
@innosmith innosmith moved this from Testable on INT to Deployed on PROD in deinklima.ch Oct 19, 2022
@innosmith innosmith moved this from Deployed on PROD to Done in deinklima.ch Oct 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for PROD Ticket ready for deployment on PROD
Projects
Development

No branches or pull requests

4 participants