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

[PE-D][Tester D] Add person command status field allowing other types than String #308

Open
nus-pe-script opened this issue Nov 8, 2024 · 0 comments

Comments

@nus-pe-script
Copy link

Describe the bug:
Add person command allows status values of non-string.

Steps to reproduce:
Enter the command for add person from the example, and change the status from recovering to other types like 12382

Expected behaviour:
A warning or error message shown

Actual behaviour:
The patient is added with the status of non-string

Screenshot:

image.png

Potential problematic behaviour:
The practitioner wouldn't know the status of the patient if the input was by accident a non-string.


Labels: type.FeatureFlaw severity.Low
original: charlenetcy/ped#9

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

No branches or pull requests

1 participant