csrinivas2130 opened a new pull request, #66508:
URL: https://github.com/apache/airflow/pull/66508
Fixes #66492
This fixes the issue Param format='time' is unpassable: UI time picker omits
seconds but jsonschema FormatChecker requires HH:MM:SS
What changed
FieldDateTime.tsx - edited (3 lines changed)
step={1} added for type="time" → browser shows the full HH:MM:SS picker
onChange pads HH:MM → HH:MM:SS for any value without seconds
value slice uses 8 chars for time, keeps 16 for everything else
FieldDateTime.test.tsx — new file created with 6 tests covering every case.
Local validation:
pnpm test FieldDateTime
pnpm lint
prek run --from-ref main --stage pre-commit
prek run --from-ref main --stage manual
all above 4 passed
Behavior impact
This does not change successful cleanup behavior.
This fix closes the issue #66492
Was generative AI tooling used to co-author this PR?
Yes — Claude Sonnet 4.6
Generated-by: Claude Sonnet 4.6 following [the
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
