Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: datepicker custom id #2040

Merged
merged 2 commits into from
Nov 7, 2024
Merged

fix: datepicker custom id #2040

merged 2 commits into from
Nov 7, 2024

Conversation

lafray
Copy link
Contributor

@lafray lafray commented Nov 6, 2024

Summary

fix #2039

Examples

before
image

after
image

…ield context

changes the way the id is handled internally so that the id that comes from the useDatePicker hook
does not overwrite the one that comes from the field context or externally. Instead, the hook will
receive these other ids for correct operation

fix #2039 for dataPicker
@lafray lafray requested a review from a team as a code owner November 6, 2024 14:53
Copy link

vercel bot commented Nov 6, 2024

Deployment failed with the following error:

You don't have permission to create a Preview Deployment for this project.

View Documentation: https://vercel.com/docs/accounts/team-members-and-roles

@lucasaarcoverde lucasaarcoverde changed the title Fix/#2039 fix: datepicker custom id Nov 7, 2024
@lucasaarcoverde lucasaarcoverde merged commit 7e731c4 into main Nov 7, 2024
8 of 9 checks passed
@lucasaarcoverde lucasaarcoverde deleted the fix/#2039 branch November 7, 2024 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

date-range-picker label
2 participants