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.

Sign up for GitHub

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

Jump to bottom

fix(DatePicker): minor bug fixes and enhancements #2355

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

fix(DatePicker): minor bug fixes and enhancements #2355

wants to merge 1 commit into from

Conversation

harshcut
Copy link
Member

@harshcut harshcut commented Sep 9, 2024

Description

  • Disabled state for incomplete range on mobile
  • Ability to override old value state for controlled updates

Changes

Additional Information

Component Checklist

  • Update Component Status Page
  • Perform Manual Testing in Other Browsers
  • Add KitchenSink Story
  • Add Interaction Tests (if applicable)
  • Add changeset

@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Sep 9, 2024

⚠️ No Changeset found

Latest commit: fb6b3d6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 9, 2024

✅ PR title follows Conventional Commits specification.

@codesandbox-ci CodeSandbox CI
Copy link

codesandbox-ci bot commented Sep 9, 2024

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit fb6b3d6:

Sandbox Source
razorpay/blade: basic Configuration

@rzpcibot
Copy link
Collaborator

rzpcibot commented Sep 9, 2024

Bundle Size Report

Updated Components
Status Component Base Size (kb) Current Size (kb) Diff
⬆ Carousel, CarouselItem 6.815 6.817 +0.002 KB

Generated by 🚫 dangerJS against fb6b3d6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chaitanyadeorukhkar chaitanyadeorukhkar Awaiting requested review from chaitanyadeorukhkar chaitanyadeorukhkar will be requested when the pull request is marked ready for review chaitanyadeorukhkar is a code owner

@kamleshchandnani kamleshchandnani Awaiting requested review from kamleshchandnani kamleshchandnani will be requested when the pull request is marked ready for review kamleshchandnani is a code owner

@anuraghazra anuraghazra Awaiting requested review from anuraghazra anuraghazra will be requested when the pull request is marked ready for review anuraghazra is a code owner

@snitin315 snitin315 Awaiting requested review from snitin315 snitin315 will be requested when the pull request is marked ready for review snitin315 is a code owner

@saurabhdaware saurabhdaware Awaiting requested review from saurabhdaware saurabhdaware will be requested when the pull request is marked ready for review saurabhdaware is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@harshcut @rzpcibot