-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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(date-range-picker): input field clear error #3409
fix(date-range-picker): input field clear error #3409
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Someone is attempting to deploy a commit to the NextUI Inc Team on Vercel. A member of the Team first needs to authorize it. |
WalkthroughThe changes involve modifying the Changes
Assessment against linked issues
Tip AI model upgrade
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Files selected for processing (2)
- apps/docs/content/components/date-range-picker/presets.ts (1 hunks)
- packages/components/date-picker/stories/date-range-picker.stories.tsx (1 hunks)
Additional comments not posted (2)
apps/docs/content/components/date-range-picker/presets.ts (1)
91-91
: Good use of optional chaining to prevent errors.The change to use optional chaining (
?.
) forfocusedValue
is a good practice to handle potentialnull
orundefined
values, thereby preventing runtime errors.packages/components/date-picker/stories/date-range-picker.stories.tsx (1)
352-352
: Good use of optional chaining to prevent errors.The change to use optional chaining (
?.
) forfocusedValue
is a good practice to handle potentialnull
orundefined
values, thereby preventing runtime errors.
Closes #3388
📝 Description
When you clear any fields inside the input of
DateRangePicker
you get the following errors.Storybook
Docs
⛳️ Current behavior (updates)
Errors as shown above
🚀 New behavior
No errors
💣 Is this a breaking change (Yes/No):
📝 Additional Information
If the PR gets accepted please use my GitHub email-id ([email protected]) instead of my other email-id for the Co-authored-by: message.
Summary by CodeRabbit
date-range-picker
to handle potentialnull
orundefined
values forstart
property, ensuring more robust performance and fewer errors.