-
Notifications
You must be signed in to change notification settings - Fork 2
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
fixes #29 #41
fixes #29 #41
Conversation
Found a intermittent issue in the e2e test for about where the click trigger was not actually happening really bizarre behavior honestly. Would consistently trigger and operate correctly with Dev tools open but not without them, Could also manually force the click that was supposed to be happening that was not on every run. added the Force option on the click and seems to behave more consistently now. |
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 looks great, thank you especially for figuring out that weird cypress thing! Cypress can definitely be a bit finicky sometimes
any chance of this getting merged in? |
@Alex-Grimes Yes! Sorry- I did not have notifications set up until recently on this repo. Thank you so much! |
@all-contributors add @Alex-Grimes for design and bugs :D |
I've put up a pull request to add @Alex-Grimes! 🎉 |
fixes #29 created a 404 page and added catchall route