Skip to content

Fix ticket number comma formatting#133

Merged
dhilgaertner merged 1 commit intomainfrom
feature/crow-124-ticket-number-comma-format
Apr 8, 2026
Merged

Fix ticket number comma formatting#133
dhilgaertner merged 1 commit intomainfrom
feature/crow-124-ticket-number-comma-format

Conversation

@dhilgaertner
Copy link
Copy Markdown
Contributor

@dhilgaertner dhilgaertner commented Apr 8, 2026 •
edited
Loading

Summary

  • Fixes ticket and PR numbers displaying with locale-aware comma separators (e.g., #1,486 instead of #1486)
  • Wraps Int string interpolations with String() across 4 UI files to bypass Swift's locale formatting

Closes #124

Test plan

  • Verify ticket numbers >= 1,000 display without commas in session list
  • Verify ticket numbers display correctly in session detail view
  • Verify issue numbers display correctly on the ticket board
  • Verify PR numbers display correctly on the ticket board and review board

🤖 Generated with Claude Code

@dhilgaertner @claude
Fix ticket and PR numbers displaying with comma separators (#124)
1c71e8a 
Wrap Int interpolations with String() to prevent locale-aware formatting
that adds commas to numbers >= 1,000 (e.g., #1,486 → #1486).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@dhilgaertner dhilgaertner requested a review from dgershman as a code owner April 8, 2026 02:19
@dhilgaertner dhilgaertner merged commit 4617e8b into main Apr 8, 2026
1 check passed
@dhilgaertner dhilgaertner deleted the feature/crow-124-ticket-number-comma-format branch April 8, 2026 02:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dgershman dgershman Awaiting requested review from dgershman dgershman is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Ticket numbers in the thousands should not have comma formatting

1 participant

@dhilgaertner