-
Notifications
You must be signed in to change notification settings - Fork 3
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
Notistack popups overlapping UI elements #306
Comments
Might be easiest to just wait until the warning auto-closes after 4000ms (?) |
Given this behavior is due to querying the production API (and getting missing data in response), I think a better approach would be to mock the response to fix this bug. wdyt @surchs ? |
Hhm, yes, I agree that in this case we should mock the call because we don't want to rely on the prod API for the tests to run. How about we do this:
|
Our Auth e2e test in #300 is failing because the warning popup for "we removed some diagnostics terms" is covering the login button:
Not sure what the best fix here is. The popup is on a timeout, so we could just make the cypress test wait longer than the popup timeout. Or we could actively click the popup away as part of the test setup, but that seems a little messy.
But we need to fix this before #300 can be merged.
The text was updated successfully, but these errors were encountered: