Skip to content

fix(ui-date-input): make DateInput2 date parsing work in every locale and timezone#1999

Merged
balzss merged 1 commit into
masterfrom
fix/dateinput2-validation-error
Jun 3, 2025
Merged

fix(ui-date-input): make DateInput2 date parsing work in every locale and timezone#1999
balzss merged 1 commit into
masterfrom
fix/dateinput2-validation-error

Conversation

@balzss

@balzss balzss commented Jun 2, 2025

Copy link
Copy Markdown
Contributor

INSTUI-4571

test plan:

  • check all examples for dateinput2, they should work as expected
  • input a date manually (with keyboard) for each example and blur the input -> it should parse the date successfully
  • placeholder hints should not have any number in them, just D, M and Y letters
  • use chrome devtools and change your location (3 dots -> more tools -> sensors -> location)
  • check the component examples again with different locales
  • check if manually changing locale and timezone works as expected

@balzss balzss self-assigned this Jun 2, 2025
@balzss balzss requested review from joyenjoyer and matyasf June 2, 2025 14:36
@github-actions

github-actions Bot commented Jun 2, 2025

Copy link
Copy Markdown
PR Preview Action v1.6.1
Preview removed because the pull request was closed.
2025-06-03 15:00 UTC

@joyenjoyer joyenjoyer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested all examples, worked fine, gj

@matyasf matyasf left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice work, I could not find any bugs

@balzss balzss merged commit 7d2ed73 into master Jun 3, 2025
12 checks passed
@balzss balzss deleted the fix/dateinput2-validation-error branch June 3, 2025 15:00
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.

3 participants