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

Modify the Error in Consultation form to be self explanatory #7410

Closed
2 tasks
nihal467 opened this issue Mar 13, 2024 · 7 comments · Fixed by ohcnetwork/care#1974
Closed
2 tasks

Modify the Error in Consultation form to be self explanatory #7410

nihal467 opened this issue Mar 13, 2024 · 7 comments · Fixed by ohcnetwork/care#1974
Assignees
Labels
P1 breaking issue or vital feature urgent

Comments

@nihal467
Copy link
Member

Describe the bug

  • When we create multiple patient consultations with same IP number, it throws an "Non Fields Errors - Patient number mustbe unique within the facility"

To Reproduce
Steps to reproduce the behavior:

  1. Create consultation for multiple patient with same IP number
  2. See error

Expected behavior

  • error message should be self explanatory something like consultation with this IP/OP number already exists; it should be a unique value.

Screenshots

image

@Dareenfadel
Copy link
Contributor

@nihal467 could you assign it to me?

@hrit2773
Copy link
Contributor

@nihal467 I would like to work on this can you please assign me

@kumar11jr
Copy link

Hii @nihal467 can you assign me this issue

@nihal467 nihal467 added urgent P1 breaking issue or vital feature labels Mar 15, 2024
Copy link

⚠️ Refrain from assigning this issue to yourself if you have another P1 issue assigned that is not yet closed.

@nihal467
Copy link
Member Author

image

  • when we input duplicate OP number , the error thrown is "something went wrong" @rithviknishad

@hrit2773
Copy link
Contributor

@nihal467 I tried to create 2 patient consultations with same IP number but they were created successfully. I did not encounter any error

@rithviknishad
Copy link
Member

@hrit2773 it needs to be in the same facility too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 breaking issue or vital feature urgent
Projects
Archived in project
5 participants