Skip to content

Fix some exceptional cases #9

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Fix some exceptional cases #9

wants to merge 5 commits into from

Conversation

oleg-koval
Copy link
Owner

fixes #8

@oleg-koval oleg-koval self-assigned this Dec 11, 2017
@wh1t3cAt1k
Copy link

@oleg-koval Hey, is this fixed with a different PR, or you plan to fix it later?

@oleg-koval
Copy link
Owner Author

@oleg-koval Hey, is this fixed with a different PR, or you plan to fix it later?

Hey @wh1t3cAt1k, have some issues here with mocking the date or maybe parsing is wrong, locally tests are passing, will need to check this later on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@oleg-koval oleg-koval

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Wrong date result from 43078.416666666664 value
2 participants
@oleg-koval @wh1t3cAt1k