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): avoid unknown attribute on div #8747

Conversation

emyarod
Copy link
Member

@emyarod emyarod commented May 24, 2021

Closes #8743
related #7037

This PR avoids adding the allowInput prop as an unknown attribute on the date picker container

Testing / Reviewing

Confirm that no prop warnings are emitted after setting a value for allowInput on <DatePicker />

@netlify
Copy link

netlify bot commented May 24, 2021

✔️ Deploy Preview for carbon-elements ready!

🔨 Explore the source changes: 417df8e

🔍 Inspect the deploy log: https://app.netlify.com/sites/carbon-elements/deploys/60ad2d2532c9e50008f42fa4

😎 Browse the preview: https://deploy-preview-8747--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented May 24, 2021

✔️ Deploy Preview for carbon-components-react ready!

🔨 Explore the source changes: 417df8e

🔍 Inspect the deploy log: https://app.netlify.com/sites/carbon-components-react/deploys/60ad2d2532c9e50008f42f9c

😎 Browse the preview: https://deploy-preview-8747--carbon-components-react.netlify.app

@emyarod emyarod force-pushed the 8743-remove-allowInput-from-datepicker-container branch from 71a3e07 to 5e04fd2 Compare May 24, 2021 16:21
Copy link
Contributor

@dakahn dakahn left a comment

Choose a reason for hiding this comment

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

ran it, added the prop, saw no warning 👍🏽

@emyarod emyarod force-pushed the 8743-remove-allowInput-from-datepicker-container branch from 6d34057 to 417df8e Compare May 25, 2021 17:00
@kodiakhq kodiakhq bot merged commit 0d2f0a3 into carbon-design-system:main May 25, 2021
@emyarod emyarod deleted the 8743-remove-allowInput-from-datepicker-container branch May 25, 2021 18:05
@jnm2377 jnm2377 mentioned this pull request May 26, 2021
22 tasks
@emyarod emyarod mentioned this pull request Jun 1, 2021
87 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Warning 'allowInput' in console when use DatePicker component
3 participants