Skip to content

fix(consumptions): keep selected month urls lowercase#24

Merged
maxstue merged 1 commit into
mainfrom
feature/kijk-333
Jun 27, 2026
Merged

fix(consumptions): keep selected month urls lowercase#24
maxstue merged 1 commit into
mainfrom
feature/kijk-333

Conversation

@maxstue

@maxstue maxstue commented Jun 27, 2026

Copy link
Copy Markdown
Owner

Summary

  • Keep the consumptions month dropdown URL value on the canonical lowercase monthSchema values.
  • Continue rendering localized month labels with formatMonth.
  • Correct the month selector aria label from year to month.

Root Cause

The command item selection used the rendered month label as the selected value. For English labels this produced values like April, which failed the consumptions route search validation because the route expects lowercase enum values like april.

Validation

  • pnpm --filter kijk-client build
  • pnpm --filter kijk-client typecheck
  • pnpm --filter kijk-client fmt
  • pnpm --filter kijk-client lint
  • pnpm dlx react-doctor@latest . --verbose --diff
  • pnpm dlx fallow audit --format json --quiet --base main --explain

Refs KIJK-333

@linear-code

linear-code Bot commented Jun 27, 2026

Copy link
Copy Markdown

KIJK-333

@maxstue maxstue marked this pull request as ready for review June 27, 2026 18:02
@maxstue maxstue self-assigned this Jun 27, 2026
@maxstue maxstue merged commit 5235bd0 into main Jun 27, 2026
13 checks passed
@maxstue maxstue deleted the feature/kijk-333 branch June 27, 2026 18:04
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.

1 participant