fix: normalize timestamps to user locale with consistent formatting (fixes #27) - #66
Conversation
Changes: - helpers.ts: pass undefined to toLocaleDateString instead of hardcoded 'en-US' - page.tsx: use formatDate() instead of raw toLocaleTimeString() Fixes inconsistent timezone display by letting the browser use the user's locale and timezone automatically. Also adds date context to event timestamps (was showing only time, now shows date + time).
|
/claim #27 |
|
Hi maintainers! 👋 Just a friendly ping on this PR. It has been approved and all checks pass. The fix addresses the timezone display inconsistency in the leaderboard described in issue #27. Would love to get this merged. Thanks! |
|
🚨 @s544vrmd4t-del @rparcus — PR #66 has been clean and mergeable for days. All CI passes, the formatDate fix uses browser locale (not hardcoded en-US). Can we get this merged today so I can collect the Opire bounty? Happy to make any final tweaks if needed. |
|
☀️ Monday morning ping — PR #66 has been clean and mergeable for days. This is a simple 2-line locale fix. Can we please get this merged today? Happy to adjust anything. |
|
@elevasyncsolutions-jpg Thanks for fixing, please check the FC |
|
@Muyideen-js Conflicts resolved. Switched to |
2 participants
Summary
Fixes #27 — timezone display inconsistency.
Changes
formatDate()from hardcodeden-USlocale toundefined, so the browser uses the viewer's locale and timezone automaticallytoLocaleTimeString()withformatDate()so event timestamps show both date + time with proper localeBefore
After
Closes #27