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

Unnecessary API Calls and Re-Renders on Patient Page #9308

Closed
i0am0arunava opened this issue Dec 5, 2024 · 2 comments
Closed

Unnecessary API Calls and Re-Renders on Patient Page #9308

i0am0arunava opened this issue Dec 5, 2024 · 2 comments

Comments

@i0am0arunava
Copy link
Contributor

Describe the bug
Unnecessary API calls and re-rendering occur when entering the first digit into the PhoneNumber field. Additionally, the Patient page causes re-rendering when the search field is initially empty, and changing options also triggers a re-render.

To Reproduce
Steps to reproduce the behavior:

  1. Go to the 'Facilities' page.
  2. Then go to the 'Patient' page.
  3. Open the network tab.
  4. Put one digit inside the PhoneNumber field, change the option, or change the country code.
  5. Notice in the network tab that an unnecessary API call is being made.

Expected behavior
For null values or when changing options, the API call should not be made. This is not expected behavior.

**Screenshots **
one digit input inside phonenuberfield
Image

while changing option
Image

while changiong country code
Image

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Desktop (please complete the following information):

  • OS: windows
  • Browser chrome
  • Version - latest

Additional context
go to the facility page then come to patient page to reproduce the issue

@i0am0arunava
Copy link
Contributor Author

I would like to work on this @nihal467 @rithviknishad

@rithviknishad
Copy link
Member

Superseded by #9400

@rithviknishad rithviknishad closed this as not planned Won't fix, can't repro, duplicate, stale Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants