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 C] Same emergency contact and patient's number are allowed #202

Closed
nus-pe-bot opened this issue Nov 3, 2023 · 1 comment · Fixed by #213
Closed

[PE-D][Tester C] Same emergency contact and patient's number are allowed #202

nus-pe-bot opened this issue Nov 3, 2023 · 1 comment · Fixed by #213
Assignees
Labels
Milestone

Comments

@nus-pe-bot
Copy link

When keying in the following command: add-patient n/John Doe ic/S1111222G g/M p/123 ec/123 e/johnd@example.com a/John street, block 123, #01-01 c/pneumothorax b/O+ t/Low, with the same phone number (123) as emergency contact number (123), the entry went through.
Refer to patient 13.
image.png

Note: Incorrect number of digits for phone number is addressed in Issues 6,7,8.

I believe this to be a feature flaw, as the patient should always put in a different emergency contact for their own safety in the future.


Labels: type.FeatureFlaw severity.High
original: maypfv/ped#9

@chonguschonguschongus chonguschonguschongus added the priority.Medium Do when have time. label Nov 5, 2023
@chonguschonguschongus
Copy link

not sure if we want to disallow this. what if the guy is just alone...

@cmHuang777 cmHuang777 self-assigned this Nov 8, 2023
@cmHuang777 cmHuang777 added this to the v1.4 milestone Nov 8, 2023
@cmHuang777 cmHuang777 mentioned this issue Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants