Skip to content
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

Open
bbmcmann opened this issue Apr 30, 2023 · 7 comments
Open

Overall User Testing #142

bbmcmann opened this issue Apr 30, 2023 · 7 comments
Assignees
Labels
bug Something isn't working

Comments

@bbmcmann
Copy link
Collaborator

bbmcmann commented Apr 30, 2023

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.

@bbmcmann bbmcmann added the bug Something isn't working label Apr 30, 2023
@bbmcmann
Copy link
Collaborator Author

Testing Windows 10 Chrome
Bugs fixing: no navbar on schedule, extra padding on shift popup, prop warnings in console

@bbmcmann bbmcmann mentioned this issue May 28, 2023
@bbmcmann bbmcmann self-assigned this May 28, 2023
@bbmcmann
Copy link
Collaborator Author

Bug fixed: no navbar on non validated page, so users cannot sign out

@bbmcmann
Copy link
Collaborator Author

bbmcmann commented May 30, 2023

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

@hopieyimmie
Copy link

hopieyimmie commented Jun 3, 2023

Bug related to Ben's comment above:
I made a test account: [email protected] and switched into my admin account to delete that test account completely. After signing out of my admin account, I tried to log in as [email protected] and get a banner saying that my account was deleted and to wait 30 days (which is good, that was my intention). However when I clicked on OK, I was still signed in as my test account and wasn't deleted from firebase. The contact page does not have the test account so it's deleted there but not in firebase. I'm not really which account I am in and not sure how to check, but it's definitely not an admin account.
Screen Shot 2023-06-03 at 12 48 21 PM
Screen Shot 2023-06-03 at 12 49 03 PM

Used Chrome

@javalosr2004
Copy link

Don't see any bugs except the ones already mentioned, and the delete feature that is currently in development.

@spagdoon0411
Copy link

spagdoon0411 commented Jun 4, 2023

Testing on MacOS Ventura (Chrome and Safari) and Windows 10 (Chrome, Edge, and Firefox).

Bugs/issues found so far:

  1. (Minor): Calendar becomes very cramped with small window sizes/resolutions; both dimensions are resized, maintaining proportions, even when there's enough space in the vertical/horizontal direction.
  2. (Minor): Calendar doesn't change to a day view when a window is resized unless the site is reloaded.
  3. (Pending review by Hospice reps): It's not obvious what the buttons at the top of a discussion-post-writing window do (they entirely replace the field text with the button text) until the draft is posted.

Planning to fix:

  • 1, the calendar resizing issue. Hoping to change containing flexbox properties or component properties so that only the necessary dimension changes. There's probably a CSS option that allows the flexbox to fill the window or something but the issue is finding which container needs to have this property changed.
  • 3, the issue with discussion-post buttons. Hoping to turn them into options for what text prepends the draft text. For example, if the Vigil Complete button is pressed, the text "Vigil Complete - " is prepended to the user's draft. If the "Vigil On Hold" button is pressed after that, that text is replaced with "Vigil On Hold - ". If the same button is pressed twice, the current prepended text should be cleared.

@javalosr2004
Copy link

javalosr2004 commented Jun 6, 2023

Testing on MacOS Ventura (Chrome 113.0.5672.92 and Firefox 113.0.2)

Issues

  • Cannot export csv of discussion, error is raised
  • (Waiting for approval) Delete shift function pops up with gray overlay from Modal.
  • Blank name can be inputted for discussion, don't know if this is a feature, if so css should be fixed for smaller devices.

Fixed

  • Resize issue (branch: resize_issue)
  • Delete function (branch: delete_shift)
  • Tab functionality when creating shift (branch: reformat_signup_form)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants