- 🇺🇸United States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.
Using the test case that was added.
Can confirm the issue in question is fixed. When I don't include seconds it auto adds 00
It seems to be fixing but would like to have a followup to update the widget so that the seconds option isn't there.
- 🇧🇪Belgium BramDriesen Belgium 🇧🇪
Hiding patch #64 as it's incomplete.
@Deshna Chauhan Please stop uploading incomplete and broken re-roll patches without an interdiff.
- 🇮🇹Italy apaderno Brescia, 🇮🇹
@Deshna When you notice that you patch is 1.66 KB instead of 6.79 KB a previous patch is, you should wonder if your patch does not miss some parts.
It happened to me too, and I learned to first check my patch size, before uploading it. - 🇳🇿New Zealand quietone
@Deshna Chauhan, I am removing credit for the unhelpful patch per How is credit granted for Drupal core issues → .
- First commit to issue fork.
- Merge request !11162Issue #2723159 : Datetime form element cannot validate when using a format without seconds → (Open) created by bhanu951
- 🇺🇸United States smustgrave
Left some comments on the MR.
Leaving the follow up tag in case that's still needed.