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

onCalendarChange returns undefined value #20557

Closed
1 task done
chinhoe opened this issue Dec 31, 2019 · 2 comments
Closed
1 task done

onCalendarChange returns undefined value #20557

chinhoe opened this issue Dec 31, 2019 · 2 comments
Assignees
Labels
3.x 3.x stable issue or bug fix, PR should be cherry-pick to master 🐛 Bug Ant Design Team had proved that this is a bug. 👷🏻‍♂️ Someone working on it

Comments

@chinhoe
Copy link

chinhoe commented Dec 31, 2019

  • I have searched the issues of this repository and believe that this is not a duplicate.

Reproduction link

Edit on CodeSandbox

Steps to reproduce

  1. Select any date in RangePicker
  2. Check returned value in console

What is expected?

Return an array of date in strings, similar to onChange

What is actually happening?

Returns 'undefined' instead

Environment Info
antd 3.26.5
React 16.12.0
System macOS Catalina (10.15.2)
Browser Google Chrome (79.0.3945.88)
@yoyo837 yoyo837 added 🐛 Bug Ant Design Team had proved that this is a bug. 3.x 3.x stable issue or bug fix, PR should be cherry-pick to master 👷🏻‍♂️ Someone working on it labels Jan 15, 2020
@yoyo837
Copy link
Contributor

yoyo837 commented Jan 15, 2020

@zombieJ 根据 react-component/calendar 的git历史,onCalendarChange 从未支持过第二个参数,文档和类型从 #8355 加入,#17373 补充其版本since 3.10.9 也是不正确的。现在是修复这个问题还是从文档中删除呢?

@zombieJ
Copy link
Member

zombieJ commented Jan 15, 2020

如果一直没有的话就清理一下文档吧。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.x 3.x stable issue or bug fix, PR should be cherry-pick to master 🐛 Bug Ant Design Team had proved that this is a bug. 👷🏻‍♂️ Someone working on it
Projects
None yet
Development

No branches or pull requests

4 participants