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

Cant add - to medication name #1

Open
Nid21cs opened this issue Nov 17, 2023 · 1 comment
Open

Cant add - to medication name #1

Nid21cs opened this issue Nov 17, 2023 · 1 comment

Comments

@Nid21cs
Copy link
Owner

Nid21cs commented Nov 17, 2023

Cant add - to medicine name. May exclude some medications.

image.png

@soc-pe-bot
Copy link

soc-pe-bot commented Nov 20, 2023

Team's Response

Accepted, but set as severity.Low.

Reason for severity.Low is that dashes appear in very rare situations as most medications have a single word as a name. Furthermore, it only causes a minor inconvenience as they can just use spacing, or even camelCase if they would like. Unlikely to affect normal app operations.

Here are some examples.

image.png

Have attached the criteria here for severity.Low
image.png

Items for the Tester to Verify

❓ Issue severity

Team chose [severity.Low]
Originally [severity.Medium]

  • I disagree

Reason for disagreement: For any users that have the medication with a - in the name, it will be inconvenient for them throughout use of the application. For example, if a user who used a medicine such as Ibuprofen-Lysine, they would have to change the way they added the medicine, and from then on, if they would like to use the find command to check on its stock or expiry date for example, they would have to remember how they added it and using the correct name would result in no medications being found.


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants