CodeIssuesPull RequestsActionsSecurityInsights
✨ AI
More
Settings

fix(dashboard): the greeting tells the visitor's time, not the server's #5529

Merged⚡ AI-generatedXSccantynz wants to mergefix/greeting-local-timemainopened 7d ago
ccantynzcommented 7d ago

"Welcome everybody by user name" — done right

The dashboard already greeted by username — but the time-of-day half computed new Date().getHours() server-side in UTC, so the owner's 9am in New Zealand was greeted with "Good evening," and so was nearly every user on earth not living near Greenwich. Warmth that's wrong is worse than none (Homecoming pillar 3 meets pillar 2).

The server now renders a timezone-neutral "Welcome back, @username" and a pre-paint inline script (same house pattern as the theme switch) swaps in the greeting computed in the visitor's clock — Good morning / afternoon / evening / Late night, each user in their own day.

Sleep-mode digest greetings (which are correctly user-hour-scheduled) untouched — their 17 tests pass. Typecheck clean.

🤖 Generated with Claude Code

gluecron[bot]🤖 botAI Reviewcommented 7d ago

AI review unavailable

The platform's AI balance is exhausted, so AI generation is temporarily unavailable. Nothing was lost. You can queue this as a repair for the internal agent from the repository's Health page, or try again once the balance is restored. The PR is otherwise unchanged.

ccantynzAI Reviewcommented 7d ago

AI Triage

(no summary)

Priority: medium Risk area: mixed

Suggested labels: (no label suggestions) Suggested reviewers: (no reviewer suggestions)

Suggestions only — nothing has been applied. The PR author stays in control.

Cross-repo impact

See what breaks downstream if this PR merges.

Analyze →
⮌ Merged

This pull request was merged into main.

c comment · e edit title · m merge · a approve · r request changes · ? shortcuts