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

When the patient has no rhythm the card is fully empty #5468

Closed
patelaryan7751 opened this issue May 17, 2023 · 9 comments · Fixed by #5641
Closed

When the patient has no rhythm the card is fully empty #5468

patelaryan7751 opened this issue May 17, 2023 · 9 comments · Fixed by #5641
Assignees

Comments

@patelaryan7751
Copy link
Contributor

Describe the bug
When the patient has no rhythm the card is fully empty

To Reproduce
Steps to reproduce the behavior:

  1. Go to https://care.coronasafe.in/facility/0c95c7f0-e1d2-4aff-83fa-933cef60d3a8/patient/dd001297-f509-4cdf-b1e8-7986ab5c7ced/consultation/1f8c13b9-aabb-4ae6-8662-5c6ff4b906bd/summary
  2. Scroll down to Rythm Card
  3. See the error

Expected behavior
There should be a message that no rhythm information is available.

Screenshots
If applicable, add screenshots to help explain your problem.
image

@patelaryan7751 patelaryan7751 self-assigned this May 17, 2023
@nikhilagastya
Copy link
Contributor

hey @patelaryan7751 can you assign me this issue !

@github-actions
Copy link

github-actions bot commented Jun 9, 2023

Hi, @gigincg, @nihal467, @khavinshankar, @mathew-alex, @aparnacoronasafe, This issue has been automatically marked as stale because it has not had any recent activity.

@nikhilagastya
Copy link
Contributor

@patelaryan7751 can you assign me this issue

@nikhilagastya
Copy link
Contributor

hey @patelaryan7751 where will I find the Rhythm component in the code base ?Can you mention the file name

@patelaryan7751
Copy link
Contributor Author

@nikhilagastya
Copy link
Contributor

thanks @patelaryan7751

@nikhilagastya
Copy link
Contributor

image
hey @patelaryan7751 does this work ?

@rithviknishad
Copy link
Member

Yes!

@nikhilagastya
Copy link
Contributor

@rithviknishad can you review the PR

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.

3 participants