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

[NO JIRA]: Fix incorrect custom prop type in calendar #2308

Merged
merged 2 commits into from
Dec 9, 2021

Conversation

olliecurtis
Copy link
Member

Fixed an issue whereby the custom SelectionConfiguration prop type was failing due to incorrect typing of type variable in the object. This change correctly types the field to use a specific value as per the prop-types spec.

Which would throw a console error in the browser of Warning: Failed prop type: checker is not a function

Remember to include the following changes:

Copy link
Contributor

@sogame sogame left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions
Copy link

github-actions bot commented Dec 7, 2021

Warnings
⚠️

Package source files (e.g. packages/package-name/src/Component.js) were updated, but snapshots weren't. Have you checked that the tests still pass?

Browser support

If this is a visual change, make sure you've tested it in multiple browsers,
particularly IE11.

Generated by 🚫 dangerJS against 99718e1

@github-actions
Copy link

github-actions bot commented Dec 7, 2021

Visual regression tests passed 😎. Bear in mind that they only run in Chromium on static components – they aren't perfect.

@github-actions
Copy link

github-actions bot commented Dec 7, 2021

Visit https://backpack.github.io/storybook-prs/2308 to see this build running in a browser.

@github-actions
Copy link

github-actions bot commented Dec 9, 2021

Visual regression tests passed 😎. Bear in mind that they only run in Chromium on static components – they aren't perfect.

@github-actions
Copy link

github-actions bot commented Dec 9, 2021

Visit https://backpack.github.io/storybook-prs/2308 to see this build running in a browser.

@olliecurtis olliecurtis merged commit 4fc607a into main Dec 9, 2021
@olliecurtis olliecurtis deleted the no-jira-calendar-prop-type-fix branch December 9, 2021 14:51
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.

2 participants