Closed romanstetsyk closed 1 year ago
LGTM, should we add testing to accompany this?
Howdy @romanstetsyk did you get a chance to review my question?
@Caleb-Cohen I'm sorry, I didn't see your last message. It's a good idea, but I still need to figure out how to write tests in Cypress. It would be great if someone could add tests to this pr or create a separate issue
@Caleb-Cohen I'm sorry, I didn't see your last message. It's a good idea, but I still need to figure out how to write tests in Cypress. It would be great if someone could add tests to this pr or create a separate issue
I'll take it up!
Description
This PR prevents adding duplicate events to the state if a recurring event spans more than 1 month.
Type of change
Please select everything applicable. Please, do not delete any lines.
Issue
Checklist:
npm run test
and all tests have passed successfully or I have included details within my PR on the failure.npm run lint
and resolved any outstanding errors. Most issues can be solved by executingnpm run format