Skip to content

fix(slack): shorten poll grid day headers to Mon Oct 5 - #104

Merged
masaok merged 1 commit into
mainfrom
fix/short-grid-day-headers
Oct 6, 2026
Merged

masaok merged 1 commit into
mainfrom
fix/short-grid-day-headers

Conversation

@masaok

@masaok masaok commented Oct 6, 2026

Copy link
Copy Markdown
Owner

Why

The poll table's day headers read "Monday, October 5th". Each day column is as wide as that string, so a poll of a few days is mostly header.

Scope

src/slack/pollMessage.ts: the header cell is a bold weekday (EEE, organizer's zone) followed by a {date_short} date element with fallback MMM d. Test and snapshot follow.

Blast radius

The channel poll message's header row only. Slack has no short weekday token, so the weekday is no longer viewer-local. A viewer far enough from the organizer that a day's first slot lands on another date sees the organizer's weekday beside their own date.

Verification

pnpm verify exit 0, 268 tests. pnpm render:fixture three-day prints the header cell as text "Tue " plus a date element with format {date_short} and fallback "Oct 6". Not yet seen drawn in Slack. That {date_short} draws as "Oct 9" comes from #97.

https://claude.ai/code/session_01WabV7wsgA4PUbXohCCgTBZ

{date_long} draws "Monday, October 5th", which makes every day column as wide as
that string. Slack has no short weekday token, so the weekday is text in the
organizer's zone and {date_short} supplies the date in the viewer's zone.

Claude-Session: https://claude.ai/code/session_01WabV7wsgA4PUbXohCCgTBZ
Copilot AI balanced review requested due to automatic review settings October 6, 2026 03:44
@vercel

vercel Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
meetmouse Ready Ready Preview Oct 6, 2026 3:45am UTC

Request Review

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@masaok
masaok merged commit 66c9465 into main Oct 6, 2026
9 checks passed
@masaok masaok mentioned this pull request Oct 6, 2026
masaok added a commit that referenced this pull request Oct 6, 2026
Carries the short poll grid day headers (#104). No change to exports.

Claude-Session: https://claude.ai/code/session_01WabV7wsgA4PUbXohCCgTBZ
@masaok
masaok deleted the fix/short-grid-day-headers branch October 6, 2026 03:50

This branch was successfully deployed

1 active deployment
Preview — 168429bc Deployed Oct 6, 2026 by vercel[bot]
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