Falsches benutzerdefiniertes Datum und falsche Uhrzeit in Warnungen bei Stummschaltung und Aussetzung

Hallo,

Mir ist aufgefallen, dass die Warnungen und E-Mails für „suspend“ und „silenced“ das falsche Datum enthalten. Entschuldigung, falls ich etwas übersehen habe.

Reproduktion

  1. Richten Sie ein (benutzerdefiniertes Datum und eine benutzerdefinierte Uhrzeit) im Modal „suspend“ ein. Ich habe eine Suspendierung für eine Woche eingerichtet.
  2. Geben Sie einen Grund und eine benutzerdefinierte E-Mail-Nachricht an.
  3. Klicken Sie auf die Schaltfläche „suspend“.

In der E-Mail steht bis zum 12. Mai 2022 um 22:00 Uhr.

Auf der Benutzerkarte wird bis zum 13. Mai 2022 um 00:00 Uhr angezeigt.


Dasselbe auf der Admin-Seite.
Screenshot 2022-05-06 at 10.13.22

Ich denke, auf der Benutzerkarte und im Admin-Bereich sollte die Uhrzeit 9:32 Uhr sein und in der E-Mail sind sowohl Datum als auch Uhrzeit falsch.

Danke :slightly_smiling_face:

3 „Gefällt mir“

Ich habe damit ein wenig herumgespielt und es scheint einige Diskrepanzen zwischen den eingegebenen Zeiten/Daten und den angezeigten Zeiten/Daten zu geben.

Bei einer Aussetzung von 1 Tag (die voreingestellte Standardeinstellung):





Und für eine benutzerdefinierte Aussetzung von 1 Woche:




In beiden Fällen war die E-Mail eine Stunde daneben, und jede Aussetzung, die länger als ein Tag dauerte, rundete die Zeit auf Mitternacht und verschob auch einen Tag vom Datum in der E-Mail-Version.

Bearbeiten: Ich habe darauf geachtet, die richtigen Screenshots zu machen, was mein Denken getrübt hat. :slightly_smiling_face:

Es liegt daran, dass 00:00 Uhr eine Stunde abgezogen wird, was dazu führt, dass es einen Tag zu früh ist. Es geht also mehr um die verlorene Stunde zwischen Eingabe und E-Mail. Und @Don hat zwei verloren. :thinking:

3 „Gefällt mir“

Ich kann alle Probleme reproduzieren. Ich werde es mir ansehen.

3 „Gefällt mir“

Dies behebt das Datum in E-Mails sowohl beim Stummschalten als auch beim Sperren von Benutzern. Daten werden in die Zeitzone des Benutzers konvertiert:

Als Nächstes müssen wir das Rundungsproblem beheben. Ich kümmere mich darum.

3 „Gefällt mir“

Der Rundungsfehler wurde gerade von @j.jaffeux in FIX: changing date should recompute input by jjaffeux · Pull Request #16937 · discourse/discourse · GitHub behoben.

3 „Gefällt mir“

Dieses Thema wurde nach 3 Tagen automatisch geschlossen. Neue Antworten sind nicht mehr möglich.