Skip to content

fix: issue happening on 31st #56

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

Merged
merged 1 commit into from
Nov 1, 2019
Merged

fix: issue happening on 31st #56

merged 1 commit into from
Nov 1, 2019

Conversation

tgpetrov
Copy link
Contributor

@tgpetrov tgpetrov commented Nov 1, 2019

PR Checklist

What is the current behavior?

If you run the demo on 31st of October and select a date from the min max section without changing it, it is off by one month.

What is the new behavior?

If you run the demo on 31st of October and select a date from the min max section without changing it, it is correct.

@tgpetrov tgpetrov requested a review from a team November 1, 2019 08:12
@DimitarTodorov
Copy link
Contributor

@cla-bot check

@cla-bot cla-bot bot added the cla: yes label Nov 1, 2019
@cla-bot
Copy link

cla-bot bot commented Nov 1, 2019

The cla-bot has been summoned, and re-checked this pull request!

@tgpetrov tgpetrov merged commit a339515 into master Nov 1, 2019
@tgpetrov tgpetrov deleted the tgpetrov/min-max-issue branch November 1, 2019 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants