Skip to content

[1141] Add locale-aware display date adapter - #2743

Open
Chris0Jeky wants to merge 2 commits into
mainfrom
issue-1141/date-adapter
Open

[1141] Add locale-aware display date adapter#2743
Chris0Jeky wants to merge 2 commits into
mainfrom
issue-1141/date-adapter

Conversation

@Chris0Jeky

Copy link
Copy Markdown
Owner

Summary

  • add a pure adapter for locale-explicit instant date, date-time, and time formatting
  • keep persisted YYYY-MM-DD calendar keys on the existing UTC-safe due-date path
  • return null for absent, invalid, or calendar/instant type-mismatch input

Proof

  • red-first focused run failed before the new utility existed with the expected unresolved import
  • focused display-date suite: 12 passed
  • existing due-date suite with the adapter: 21 passed
  • explicit en/it/es locale, leap-day, invalid-input, and UTC/Pacific near-midnight coverage
  • npm run lint (8 pre-existing warnings, 0 errors)
  • npm run typecheck
  • npm run build (known ineffective dynamic-import warning only)
  • git diff --check

Refs #1141

No issue close or merge requested; this PR is intentionally left open for review.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

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

Labels

None yet

Projects

Status: Pending

Development

Successfully merging this pull request may close these issues.

1 participant