-
Notifications
You must be signed in to change notification settings - Fork 36
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
Possible bug: keeps refreshing after date selection #28
Comments
@jvolker I am thinking we need to add automated unit testing. I've been looking into it but we may want to change the class on a date to simulate a date being available. |
Yes, this sounds like a great idea. I don't have a lot of knowledge about testing though. Would you be able to set up a test environment and that particular unit test? |
@jvolker I just changed line 115 to |
@jvolker I'm not able to reproduce this issue on the book assistance. Maybe it was for the chrome extension? |
Thanks for checking @alexDrinkwater. I guess that's a good thing. :)
@sankeerthgone could you please confirm what part of the software you were using - MIQ Booking Assistance (Node.js script) or MIQ Alert (Chrome browser extension)? |
Unable to reproduce. |
Posted in #25 by @sankeerthgone
@alexDrinkwater do you have a recommendation on how to debug/test this? It seems free spots and therefore alerts are rarer these days. Can we mimic it somehow?
The text was updated successfully, but these errors were encountered: