[8440] Issues with Time/Region controllers on Publish For Later

[jvega190]

• Fixed the selector accepting the following input 11/14/2025 hh:mm PM as valid.

craftercms/craftercms#8440

Summary by CodeRabbit

• Bug Fixes
  • Improved date/time/timezone picker to prevent redundant updates and ensure only valid, substantive changes are processed.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

The PR modifies the handleChange function in DateTimeTimezonePicker.tsx to validate and compare new date values before updating state. It adds a guard that checks if the new value is valid and differs from the current date's ISO string representation before invoking state updates.

Changes

Cohort / File(s)	Summary
DateTime Picker Logic ui/app/src/components/DateTimeTimezonePicker/DateTimeTimezonePicker.tsx	Modified handleChange to guard setSelectedDate with validity check and ISO string difference comparison, preventing unnecessary updates from duplicate or invalid values.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related issues

• Issues with Time/Region controllers on Publish For Later craftercms#8440: This PR directly addresses the DateTimeTimezonePicker's handleChange logic by tightening the conditions for when state updates occur, which relates to the time/region controller behaviors described in the issue.

Suggested reviewers

• rart
• sumerjabri

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly references the issue number [8440] and summarizes the main change involving Time/Region controllers, which aligns with the bugfix for the datetime picker validation logic.
Description check	✅ Passed	The description includes a clear fix summary and references the GitHub issue, meeting the template requirements of ticket reference or full description of what's in the PR.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}