-
Notifications
You must be signed in to change notification settings - Fork 0
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
Overall User Testing #142
Comments
Testing Windows 10 Chrome |
Bug fixed: no navbar on non validated page, so users cannot sign out |
New bug found: If a user is rejected via admin validation page, then tries to sign up again with the same email, they get an error that the email is already in use, when it is not. This could be an issue if they are mistakenly rejected or rejected because they typed their name wrong or something. While unlikely, this seems like a possible use path. We already remove their user document, now somehow need to remove user from Authentication in firebase as well. Alternatively, we could not authenticate users until validated. I'm not sure which is harder but removing authentication seems easier this is LOW PRIORITY |
Bug related to Ben's comment above: Used Chrome |
Don't see any bugs except the ones already mentioned, and the delete feature that is currently in development. |
Testing on MacOS Ventura (Chrome and Safari) and Windows 10 (Chrome, Edge, and Firefox). Bugs/issues found so far:
Planning to fix:
|
Testing on MacOS Ventura (Chrome 113.0.5672.92 and Firefox 113.0.2)Issues
Fixed
|
Please try to break the site. Put in bad inputs into forms, try to use incorrect/invalid dates, think of common mistakes users might make, attempt to inject code, use different internet strengths, try to test on different operating systems, test different window sizes.
Anything you can think of that may break the site, test it. When you can, have the dev console open so you can see if there are errors not seen by users (maybe a bad request).
If you are on this task, comment what OS and browsers (try to use multiple) you are going to test on.
If you find a bug, comment it here.
Then, see if you can fix it on it's own branch.
Update your comment to note if you did/did not fix it.
The text was updated successfully, but these errors were encountered: