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

JS Warnings when changing the publish date #10876

Closed
afercia opened this issue Oct 21, 2018 · 8 comments
Closed

JS Warnings when changing the publish date #10876

afercia opened this issue Oct 21, 2018 · 8 comments
Labels
Good First Issue An issue that's suitable for someone looking to contribute for the first time Needs Dev Ready for, and needs developer efforts [Package] Components /packages/components [Type] Bug An existing feature does not function as intended

Comments

@afercia
Copy link
Contributor

afercia commented Oct 21, 2018

Describe the bug
Not sure this is already tracked. If it's a known issue, of course do feel free to close.

On current master:

  • create a post
  • click the Publish date button and observe your browser's dev tools console
  • as soon as the date picker opens, I'm getting this JS Warning:

Warning: Failed prop type: DayPickerSingleDateController: unknown props found: block

@afercia afercia added the [Type] Bug An existing feature does not function as intended label Oct 21, 2018
@desrosj
Copy link
Contributor

desrosj commented Oct 25, 2018

@afercia I am unable to reproduce this. Can you give me more details about which WP version, Gutenberg version, and date format the site has selected?

@afercia
Copy link
Contributor Author

afercia commented Oct 26, 2018

@desrosj this was tested on WP trunk and Gutenberg master 5 days ago, default date format F j, Y g:i a. Cannot reproduce now, however I'm not sure DayPickerSingleDateController has a block prop, see https://github.com/airbnb/react-dates/blob/60025377603ac1bb042b525d499ea0d5dc814e89/src/components/DayPickerSingleDateController.jsx

I see this props is passed to the component though:

<DayPickerSingleDateController
block

@Ferdev
Copy link

Ferdev commented Jan 14, 2019

we're also seeing this issue in https://github.com/woocommerce/woocommerce-bookings
We're just using the DatePicker class from @wordpress/components, and we get the warning whenever we run the plugin under WordPress 5.0.3 (it does work in 5.0.2 and previous versions).

As @afercia pointed out, I don't think DayPickerSingleDateController has a block property, but it does have a blocked property I think... Maybe it was just typo.

We'd appreciate any info you could give us, thanks!

Steps to reproduce the issue:

  1. WordPress 5.0.3 fresh install
  2. Install & activate the WooCommerce plugin.
  3. Install & activate the WooCommerce Bookings plugin.
  4. Execute npm run start on the WooCommerce Bookings plugin folder
  5. In the Admin, go to Bookings->Calendar an click on the month name on the calendar's title.
  6. The error should pop up in the browser's dev console.

@afercia
Copy link
Contributor Author

afercia commented Jan 14, 2019

Yep, I still see it on current master 16b94bb when opening the Publish date picker.

@bor0
Copy link
Contributor

bor0 commented Aug 19, 2019

I can't reproduce this on WordPress 5.2.2.

@gziolo
Copy link
Member

gziolo commented Oct 21, 2019

I'm seeing this React deprecations listed:
Screen Shot 2019-10-21 at 11 15 10

@gziolo gziolo added [Package] Components /packages/components Good First Issue An issue that's suitable for someone looking to contribute for the first time Needs Dev Ready for, and needs developer efforts labels Oct 21, 2019
@glendaviesnz
Copy link
Contributor

I am not seeing any console errors with the date picker in WordPress 5.3 with Gutenberg 7.0.0

date-picker

@afercia
Copy link
Contributor Author

afercia commented Nov 29, 2019

Looks like the block property was removed in #12933 thus fixing this issue.

@afercia afercia closed this as completed Nov 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good First Issue An issue that's suitable for someone looking to contribute for the first time Needs Dev Ready for, and needs developer efforts [Package] Components /packages/components [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

6 participants