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(date picker): detect change range interval console error #11532

Conversation

sangeethababu9223
Copy link
Member

Related Ticket(s)

Closes #11490

Description

In Date picker element, dev tool console shows error Uncaught TypeError: fp.getDateStr is not a function.

Changelog

Removed

  • Removed custom event handling function.

@sangeethababu9223 sangeethababu9223 requested a review from a team as a code owner February 13, 2024 07:01
@sangeethababu9223 sangeethababu9223 requested review from kennylam and emyarod and removed request for a team February 13, 2024 07:01
@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Feb 13, 2024

@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Feb 13, 2024

@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Feb 13, 2024

@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Feb 13, 2024

Copy link
Contributor

@m4olivei m4olivei left a comment

Choose a reason for hiding this comment

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

Looks like what happened between when I last worked on this up until now is that, v2 was merged and it reverted flatpickr from the version I had updated it to, 4.6.13, back to 4.1.6.

I don't have a strong opinion, ultimately we'll likely want to drop flatpickr eventually b/c it's not well maintained, but another option is to set flatpickr to 4.6.13 (note we have patches to make it all work). Once set to 4.6.13, everything should work the same as v1.

If we keep this change as is, which I kinda like b/c less code we're maintaining, don't need patches, and seems to work well, we should remove the patch for 4.6.13.

Copy link
Contributor

@m4olivei m4olivei left a comment

Choose a reason for hiding this comment

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

LGTM!

Thanks for the changes @sangeethababu9223 .

@kennylam kennylam added Ready to merge Label for the pull requests that are ready to merge package: carbon web components labels Feb 27, 2024
@kodiakhq kodiakhq bot merged commit eee2c73 into carbon-design-system:main Feb 28, 2024
12 of 19 checks passed
Copy link
Contributor

Hey there! This issue/pull request was referenced in recently released v2.5.0.

@IgnacioBecerra IgnacioBecerra self-assigned this Mar 13, 2024
IgnacioBecerra pushed a commit to kennylam/carbon-for-ibm-dotcom that referenced this pull request Mar 13, 2024
…design-system#11532)

Closes carbon-design-system#11490

In Date picker element, dev tool console shows error `Uncaught TypeError: fp.getDateStr is not a function`.

**Removed**

- Removed custom event handling function.

<!-- React and Web Component deploy previews are enabled by default. -->
<!-- To enable additional available deploy previews, apply the following -->
<!-- labels for the corresponding package: -->
<!-- *** "test: e2e": Codesandbox examples and e2e integration tests -->
<!-- *** "package: services": Services -->
<!-- *** "package: utilities": Utilities -->
<!-- *** "RTL": React / Web Components (RTL) -->
<!-- *** "feature flag": React / Web Components (experimental) -->
IgnacioBecerra pushed a commit to kennylam/carbon-for-ibm-dotcom that referenced this pull request Mar 13, 2024
…design-system#11532)

Closes carbon-design-system#11490

In Date picker element, dev tool console shows error `Uncaught TypeError: fp.getDateStr is not a function`.

**Removed**

- Removed custom event handling function.

<!-- React and Web Component deploy previews are enabled by default. -->
<!-- To enable additional available deploy previews, apply the following -->
<!-- labels for the corresponding package: -->
<!-- *** "test: e2e": Codesandbox examples and e2e integration tests -->
<!-- *** "package: services": Services -->
<!-- *** "package: utilities": Utilities -->
<!-- *** "RTL": React / Web Components (RTL) -->
<!-- *** "feature flag": React / Web Components (experimental) -->
kennylam pushed a commit to kennylam/carbon-for-ibm-dotcom that referenced this pull request Jun 11, 2024
…design-system#11532)

### Related Ticket(s)

Closes carbon-design-system#11490

### Description

In Date picker element, dev tool console shows error `Uncaught TypeError: fp.getDateStr is not a function`.

### Changelog

**Removed**

- Removed custom event handling function.

<!-- React and Web Component deploy previews are enabled by default. -->
<!-- To enable additional available deploy previews, apply the following -->
<!-- labels for the corresponding package: -->
<!-- *** "test: e2e": Codesandbox examples and e2e integration tests -->
<!-- *** "package: services": Services -->
<!-- *** "package: utilities": Utilities -->
<!-- *** "RTL": React / Web Components (RTL) -->
<!-- *** "feature flag": React / Web Components (experimental) -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: carbon web components Ready to merge Label for the pull requests that are ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[cds-date-picker]: Detecting change on range interval
5 participants