Skip to content

fix(property): date display#985

Merged
ehayes2000 merged 1 commit intomainfrom
fix-property-date-display
Jan 14, 2026
Merged

fix(property): date display#985
ehayes2000 merged 1 commit intomainfrom
fix-property-date-display

Conversation

@ehayes2000
Copy link
Copy Markdown
Contributor

No description provided.

@ehayes2000 ehayes2000 requested a review from a team as a code owner January 14, 2026 00:31
@github-actions
Copy link
Copy Markdown

@claude
Copy link
Copy Markdown
Contributor

claude bot commented Jan 14, 2026

Code review

Missing Test Coverage

This PR fixes a date display bug by removing timezone workaround logic (lines 24-31 in formatting.ts), but does not include tests to verify the fix or prevent regressions.

According to CLAUDE.md:

Testing - Write tests for your changes. When fixing bugs or regressions, identify the issue with a test before fixing it.

Recommendation:
Add tests for the formatDate function that:

  1. Reproduce the original bug that this fix addresses
  2. Verify the fix works correctly
  3. Cover edge cases (e.g., dates near midnight, dates in different timezones)
  4. Prevent future regressions

This will ensure the date formatting behavior is correct and maintainable going forward.

@ehayes2000 ehayes2000 merged commit 673bb95 into main Jan 14, 2026
23 checks passed
@ehayes2000 ehayes2000 deleted the fix-property-date-display branch January 14, 2026 01:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants