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

Use provided time instead of time.now in NewEventWithTime #20

Closed

andresbott wants to merge 3 commits into ZachtimusPrime:master from andresbott:master

andresbott commented Nov 13, 2020 •

edited

Loading

This pr mainly correct NewEventWithTime to use the provided time.
Also found an unhandled error.
solves: #21

bott added 3 commits

November 13, 2020 13:43


          Use provided time instead of time.now in NewEventWithTime

b7eaa4f


          handle doRequest error

791e57e


          update gitignore

7f8a520

andresbott mentioned this pull request

NewEventWithTime ignores provided timestamp #21

Open

0xd020 approved these changes

View reviewed changes

This pull request was closed.

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