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

Alert doesn't let user to fill complete information #5271

Closed
AnushaUp opened this issue Jul 25, 2023 · 3 comments
Closed

Alert doesn't let user to fill complete information #5271

AnushaUp opened this issue Jul 25, 2023 · 3 comments
Assignees
Labels
Build test: success Solution: Proposed Solution has been proposed but not agreed upon yet Status: To do

Comments

@AnushaUp
Copy link

AnushaUp commented Jul 25, 2023

Describe the bug

While creating new duplicate patient, while adding information the duplicate alert appears in the middle of information input and it disturb the form filling process and redirect back to the list and don't let fill the full information on the form.

To reproduce

Steps to reproduce the behaviour:

  1. Go to mobile site
  2. Click on Dispensing
  3. Click on New patient
  4. Try creating new duplicate patient
  5. See error

Video here

20230725_153336.mp4

Expected behaviour

Should let fill the full information

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: v8.6.6
  • Tablet model:
  • OS version:

Additional context

Noticed while testing THIS

@AnushaUp AnushaUp added the Solution: Proposed Solution has been proposed but not agreed upon yet label Jul 25, 2023
@DhanyaHerath
Copy link
Contributor

Good catch thank you @AnushaUp!

@ujwalSussol
Copy link
Contributor

@arjunSussol : have you written some code on this. Could you publish it so that folks can see progress. Perhaps you can write Not Ready for Review ... @sanjeevsussol please create a label Draft:Don't Review ... if a PR is likely to take long then we may have to use this approach so that folks in NZ know that work is being done.

sah-arjun added a commit that referenced this issue Dec 26, 2023
…-info-due-to-alert

#5271 fix annoying alert to confirm duplicate patient
@AnushaUp
Copy link
Author

AnushaUp commented Dec 27, 2023

Testing

Tested in Tested in Mobile v8.6.8-rc01

  • Dispensary > Try to create a duplicate patient having same firstName, lastName, and DoB
  • There should be alert with confirmation to create duplicate patient.
  • If click Cancel then cancel the process and return back to patient list window
  • If click OK then remains in the same window and fill the required fields then create patient. PASSED

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build test: success Solution: Proposed Solution has been proposed but not agreed upon yet Status: To do
Projects
None yet
Development

No branches or pull requests

5 participants