-
Notifications
You must be signed in to change notification settings - Fork 532
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
Redundant API calls in Vitals tab #8864
Redundant API calls in Vitals tab #8864
Conversation
- fetch pain data along with rest of the graph data and pass it as a prop to PainDiagrams - translations
✅ Deploy Preview for care-ohc ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@Jacobjeevan It's been partially fixed, but when you open the tab and refresh the page, there's still a redundant request going through. |
It was a top-level re-render (caused by patientData), added it to the other checks we have. This should take care of extra calls in other tabs as well 👍 |
LGTM |
👋 Hi, @Jacobjeevan, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
@Jacobjeevan Can you clear the conflicts |
@Jacobjeevan Your efforts have helped advance digital healthcare and TeleICU systems. 🚀 Thank you for taking the time out to make CARE better. We hope you continue to innovate and contribute; your impact is immense! 🙌 |
Proposed Changes
@ohcnetwork/care-fe-code-reviewers
Merge Checklist