-
Notifications
You must be signed in to change notification settings - Fork 5
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
Throw error on event creation if start date is later than end date #355
Conversation
Thanks for contributing! If you've made changes to the API's functionality, please make sure to bump the package version—see this guide to semantic versioning for details—and document those changes as appropriate. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is close to being done, just a few more changes as well as linting and removing unnecessary files.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few nitpicks just to make sure the code base is extreeeemely consistent.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally good, I think you will need to set up automating the test for the new test file (add it to test/index.ts). After you are done, verify that yarn test will run all the unit tests in the new test file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
…cmucsd#355) * Throw error on event creation if start date is later than end date * Test for create event with invalid date * Remove unused imports, add trailing commas, add missing semi-colon * Remove unecessary quotes * Add EOL at end * Add event testers for success and duplicate code case * fix linter issues * Permissions test for event creation * Added event creation into repo check. Some Comments * linter error fix * more linter * last check * bugfix * check event lookup * fix services: * lint * more lint * lint fix * fix moment * boolean fix
Added condition in the create function for the event service to check if the end date was later than the start date
Closes #179.