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

Pop-up for selection of facility, redundant #6518

Closed
aparnacoronasafe opened this issue Oct 30, 2023 · 4 comments · Fixed by #6535
Closed

Pop-up for selection of facility, redundant #6518

aparnacoronasafe opened this issue Oct 30, 2023 · 4 comments · Fixed by #6535
Assignees

Comments

@aparnacoronasafe
Copy link
Member

Describe the bug
When a user has access to only 1 facility, and he/she goes to patient page (https://care.coronasafe.in/patients) and tries to register a new patient by clicking on "+ Add patient details", a pop-up window to select the facility the patient is admitted to appears. Such pop-up window is redundant since the user can anyway only choose the one facility he has access to.

Desirable flow: The pop-up window for user to select facility must only appear for users that have access to more than one facility.

To Reproduce
Steps to reproduce the behavior:

  1. Login with the username "test-staff123", pass: Lilo@123
  2. Note that the user has access to only 1 facility
  3. Click on 'patients' in side nav menu
  4. Click on "+ Add patient details"
  5. See the pop-up window to select facility and the dropdown has only 1 option.

Expected behaviour
The user upon clicking "+ Add patient details" must be taken to the patient registration form, since it is obvious that the patient is being admitted to the only facility that the user has access to.

@sriharsh05
Copy link
Contributor

I want to work on this issue. Please assign this to me.

@AnuMessi10
Copy link

Hi @aparnacoronasafe, I am new to this and would like to contribute, let me know if you can assign this to me. Cheers!

@Ashesh3
Copy link
Member

Ashesh3 commented Nov 2, 2023

Hi @aparnacoronasafe, I am new to this and would like to contribute, let me know if you can assign this to me. Cheers!

Just a heads up, @sriharsh05 had already requested this issue first, so we're going to assign it to them. But there are plenty of other issues up for grabs. Thanks for understanding!

@AnuMessi10
Copy link

Hi @aparnacoronasafe, I am new to this and would like to contribute, let me know if you can assign this to me. Cheers!

Just a heads up, @sriharsh05 had already requested this issue first, so we're going to assign it to them. But there are plenty of other issues up for grabs. Thanks for understanding!

No issues, let me know if @sriharsh05 needs any help with this.

Meanwhile I will definitely explore other interesting issues 😁

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

Successfully merging a pull request may close this issue.

4 participants