pra-navi / pe

0 stars 0 forks source link

Not enough images #15

Open pra-navi opened 10 months ago

pra-navi commented 10 months ago

image.png image.png

There should be more visual aid for new users who are not tech savvy

soc-pe-bot commented 10 months ago

Team's Response

We provided screenshots for commands we deemed necessary to have images for example the view commands have a screenshot each. We feel additional images are a good to have but not a bug.

The suggestion to have more image is still valid, thus we will proceed to label it Not In Scope.

Moreover the severity of this should be VeryLow as it is simply cosmetic.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [x] I disagree **Reason for disagreement:** ![image.png](https://raw.githubusercontent.com/pra-navi/pe/main/files/2d6777c9-ed87-405e-b601-723a4c7e83c6.png) Within the current UG, there are little to no explanations using visual aids like screenshots of the app before and after commands to try to help guide new users who do not understand what the they are doing. New users will be even more lost considering how some commands have sample usage with outputs while some do not, for eg they are not sure if they are supposed to receive an output message for add_tag_p or not and a screenshot will help clarify the situation. ![image.png](https://raw.githubusercontent.com/pra-navi/pe/main/files/9534801e-dbce-46e4-9341-6f3eb14d3f5b.png) -> ![image.png](https://raw.githubusercontent.com/pra-navi/pe/main/files/c43cb845-dd6c-45e0-8925-701c5b2494c1.png) ![image.png](https://raw.githubusercontent.com/pra-navi/pe/main/files/729abae0-1a82-4826-a367-32fad5867e0b.png) -> ![image.png](https://raw.githubusercontent.com/pra-navi/pe/main/files/7f4fd0cc-291d-4ed3-b7df-ea13077a2c87.png) With a screenshot for add_a, one can show clearly the output message, and where they should see to see whether they patient list updated at the bottom of the list and annotate the newly added patient. With a screenshot for add_tag_p, one can show clearly the output message that they should expect for a correct input and that they are supposed to use `view_p` to view their added medical condition. (this is unclear from the current explanation but the screenshots would help a lot) ![image.png](https://raw.githubusercontent.com/pra-navi/pe/main/files/b6a05e4e-bcff-4234-9172-402c7c38352b.png) Since this app is a new transition for all clinic receptionists, clarity in UG is most important and this is something that will potentially affect some new users who are not technologically savvy (a cause for inconvenience) so I would like to propose the Low Severity.