Skip to content

Comments

Format time range#37

Merged
dweekly merged 2 commits intomainfrom
format-time-range
Jan 21, 2026
Merged

Format time range#37
dweekly merged 2 commits intomainfrom
format-time-range

Conversation

@mattspitz
Copy link
Collaborator

Update formatTimeRange for consistency.

Previously, we were combining meridians, using lowercase, and inconsistently using spaces. Now, we're just doing what we're doing everywhere else (namely, look at the "past shows" vs. "current shows" in the screenshots).

Before:
Screenshot from 2026-01-20 16-45-06

After:
Screenshot from 2026-01-20 16-44-46

Previously, we were combining meridians, using lowercase, and
inconsistently using spaces. Now, we're just doing what we're doing
everywhere else (namely, look at the "past shows" vs. "current shows" in
the screenshots).
@mattspitz mattspitz requested a review from dweekly January 21, 2026 00:58
@github-actions
Copy link

🎸 Sinister Dexter Preview Deployment

This build: https://c42e7969.sinister-dexter.pages.dev
🔄 Latest on PR: https://preview-37.sinister-dexter.pages.dev/

Commit: b81d961

The first link shows this exact build, while the second always shows the latest changes on this PR.

@dweekly dweekly merged commit cc77b06 into main Jan 21, 2026
1 check passed
@github-actions
Copy link

✅ FTP Deployment Successful!

PR #37: Format time range
Merged by: @dweekly
Commit: cc77b06

Deployment Details

  • Server: server298.com
  • Directory: /www/www-2025/
  • Time: Wed, 21 Jan 2026 01:20:30 GMT

🌐 Live Site: https://sinisterdexter.net/

All tests passed and the site has been successfully deployed to production.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants