fix(alerting): show local time alongside UTC - #161
Merged
Conversation
edospadoni
deployed
to
improve_lert_notification - my-backend-qa PR #161
July 28, 2026 14:33 — with 
Render
Active
Render
Active
Contributor
|
🔗 Redirect URIs Added to Logto The following redirect URIs have been automatically added to the Logto application configuration: Redirect URIs:
Post-logout redirect URIs:
These will be automatically removed when the PR is closed or merged. |
andre8244
approved these changes
Jul 29, 2026
edospadoni
temporarily deployed
to
improve_lert_notification - my-collect-qa PR #161
July 29, 2026 07:51 — with
Render
Destroyed
Render
Destroyed
gsanchietti
changed the title
gsanchietti
marked this pull request as draft
July 29, 2026 07:52
gsanchietti
changed the title
gsanchietti
marked this pull request as ready for review
July 29, 2026 07:58
gsanchietti
force-pushed
the
improve_lert_notification
branch
from
July 29, 2026 08:01
35621a6 to
ab694cc
Compare
edospadoni
deployed
to
improve_lert_notification - my-backend-qa PR #161
July 29, 2026 08:01 — with
Render
Active
Render
Active
Alert emails/Telegram messages showed StartsAt/EndsAt in UTC only, confusing recipients checking against wall-clock time. Uses Alertmanager's tz func to add Europe/Rome time (CEST/CET) next to UTC. Assisted-by: Claude Code:claude-sonnet-5
gsanchietti
force-pushed
the
improve_lert_notification
branch
from
July 29, 2026 08:26
ab694cc to
ad92d09
Compare
edospadoni
deployed
to
improve_lert_notification - my-backend-qa PR #161
July 29, 2026 08:26 — with
Render
Active
Render
Active
edospadoni
deployed
to
improve_lert_notification - my-mimir-qa PR #161
July 29, 2026 08:26 — with
Render
Active
Render
Active
Timestamps showed the local time and a parenthesised UTC duplicate on the same line, with seconds and a hardcoded Europe/Rome in every template, the English ones included. Render a single local time in the format the web UI already uses, with UTC as a secondary line below it: muted grey in the HTML bodies, indented to the value column in the plain text ones. The zone moves to ALERTING_TIMEZONE (default Europe/Rome) and is substituted like APP_URL already was, so format and zone live in one shared _datetime.tmpl instead of being repeated. Unify the label wording across the three channels: the plain text column abbreviated RIVEND./DISTRIB./ORG. to fit, the HTML said DESCRIZIONE where the others said DETTAGLI, and the Italian Telegram resolved message said "Attivo da" where it meant "Inizio". Widen the column to the longest label instead of abbreviating. Add render tests. The templates run inside Mimir's alertmanager, so nothing here executed them and a broken one surfaced only as a notification that never arrived. Drop the zoneinfo bind mount from the local Mimir compose: it runs the official image, which already ships tzdata.
edospadoni
temporarily deployed
to
improve_lert_notification - my-mimir-qa PR #161
July 29, 2026 10:38 — with
Render
Destroyed
Render
Destroyed
edospadoni
had a problem deploying
to
improve_lert_notification - my-backend-qa PR #161
July 29, 2026 10:38 — with
Render
Failure
Render
Failure
ALERTING_TIMEZONE is validated with time.LoadLocation at startup, but the runtime image is alpine without the tzdata package, so the default Europe/Rome failed to resolve and the backend refused to boot. Embed the database in the binary rather than adding the package to the image: this check is the only thing here that needs it, and embedding survives a change of base image.
edospadoni
temporarily deployed
to
improve_lert_notification - my-backend-qa PR #161
July 29, 2026 10:44 — with
Render
Destroyed
Render
Destroyed
Contributor
|
🗑️ Redirect URIs Removed from Logto The following redirect URIs have been automatically removed from the Logto application configuration: Redirect URIs:
Post-logout redirect URIs:
Cleanup completed for PR #161. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Alert emails/Telegram messages showed StartsAt/EndsAt in UTC only,
confusing recipients checking against wall-clock time. Uses
Alertmanager's tz func to add Europe/Rome time (CEST/CET) next to UTC.
Assisted-by: Claude Code:claude-sonnet-5