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

case sensitive should not be checked during creating name #5237

Closed
arjunSussol opened this issue Jun 22, 2023 · 3 comments · Fixed by #5238
Closed

case sensitive should not be checked during creating name #5237

arjunSussol opened this issue Jun 22, 2023 · 3 comments · Fixed by #5238
Assignees
Labels
Bug: development Bug was found and believed to only be in development branches Effort: small Small effort story/task, estimated 0-2 days Priority: high Status: In progress tester: Anusha

Comments

@arjunSussol
Copy link
Contributor

Describe the bug

Patient duplicity alert is not generated if the duplicate name created with the different case.
For e.g. Arjun is not equal to arjun and duplicate name confirmation alert failed.

To reproduce

name_case_sensitive_issue.mp4

Steps to reproduce the behaviour:

  1. Go to Vaccination
  2. Look for an existing name
  3. Try create duplicate name as above but with different case
  4. No duplicity check alert generated

Expected behaviour

Arjun = arjun in the case of username/name, but not valid for password.
Duplicity check should be performed without caring about the case of the firstName and lastName

Proposed Solution

Leave if you don't know how to fix/implement. Edit this issue description and explain here if you know the best path of implementing the fix within the codebase.

Version and device info

  • App version:
  • Tablet model:
  • OS version:

Additional context

Add any other context about the problem here.

@arjunSussol arjunSussol added the Bug: ???? Placeholder; replace with 'development' or 'production' label Jun 22, 2023
@arjunSussol arjunSussol self-assigned this Jun 22, 2023
@arjunSussol arjunSussol added Priority: high Effort: small Small effort story/task, estimated 0-2 days Bug: development Bug was found and believed to only be in development branches Status: In progress tester: Anusha and removed Bug: ???? Placeholder; replace with 'development' or 'production' labels Jun 22, 2023
@arjunSussol
Copy link
Contributor Author

arjunSussol added a commit that referenced this issue Jun 22, 2023
#5237 Case sensitive should be ignored to check name du
arjunSussol added a commit that referenced this issue Jul 6, 2023
…-be-ingored-for-name

#5237 Case sensitive should be ignored to check name duplicity
@arjunSussol arjunSussol linked a pull request Jul 6, 2023 that will close this issue
4 tasks
@AnushaUp
Copy link

AnushaUp commented Jul 10, 2023

Testing

Tested in v8.6.6-rc02
Steps to reproduce or otherwise test the changes of this PR:

  • Create a new name e.g. arjun sah and enter dob and other mandatory fields.
  • Again try to create same name with Caps On e.g. ARJUN SAH, enter dob as above and mandatory fields in the dispensing form and just first, last, and dob in the vaccination form.
  • There should be an alert Are you sure you want to duplicate this patient? A patient with name: ARJUN SAH and date of birth: {1} already exist.
  • Click either Cancel or Save/Next to create duplicate patient.

@AnushaUp
Copy link

@arjunSussol Tested and passed👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug: development Bug was found and believed to only be in development branches Effort: small Small effort story/task, estimated 0-2 days Priority: high Status: In progress tester: Anusha
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants