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 DatetimeField #334

Closed
wants to merge 8 commits into from
Closed

Fix DatetimeField #334

wants to merge 8 commits into from

Conversation

wisetc
Copy link
Contributor

@wisetc wisetc commented Dec 30, 2018

  • 简单修复组件 DatetimeField 跨年问题
  • 增加该组件跨年、跨月的测试,运行 npx jest

@eternalsky
Copy link
Contributor

非常感谢pr,关于测试部分,我们这里有另外的计划,因此这部分不能合并进去,不过 tc 部分我们会在测试体系建立后移动过去,感谢为 saltui 做出的贡献,这个 pr 先 close 掉了,对应的问题已有fix。

@eternalsky eternalsky closed this Jan 2, 2019
@wisetc
Copy link
Contributor Author

wisetc commented Jan 2, 2019

感谢及时修复,衷心希望saltui不断完善,越来越好用,受到更多人关注和肯定。

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