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

Added explicit precision for date and time in event bookings #246

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mitchellkwong
Copy link
Contributor

Description of Change

Explicitly specified precision when comparing dates and times for event booking input validation.

Link To Issue

Resolves an issue where dates were inaccurately flagged as invalid.
image

How to test this change

Tested on a local build.

Checklists

Checklist for developer of feature

  • I have performed a code review on my own code
  • I have tested my code against the Travis CI, and it has passed.
  • I have linked to the appropriate issue on GitHub
  • I have written an appropriate description for this PR.
  • I have written how to test this PR.

Checklist for reviewers

  • I have reviewed the new code
  • I have followed the given steps to test the PR, and it passes.

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.

1 participant