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

Bug: Medicine Administrations table starts with an additional hour offset to the future #6323

Closed
rithviknishad opened this issue Sep 22, 2023 · 3 comments
Assignees
Labels
bug Something isn't working good first issue Good for newcomers P2

Comments

@rithviknishad
Copy link
Member

Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

  1. Go to Medicine Administrations Table

Expected behavior
The additional 1-hour column (where administrations cannot be made at the present time) should not be present.

Screenshots
image

Reported by @nihal467

@rithviknishad rithviknishad added bug Something isn't working good first issue Good for newcomers P2 labels Sep 22, 2023
@print-Sathvik print-Sathvik self-assigned this Sep 23, 2023
@print-Sathvik
Copy link
Contributor

End date: 23/09/2023

@rithviknishad
Copy link
Member Author

The latest date should always be the latest of all last administered date.

i.e. Medicine A is last administered 23rd 10:02 AM, Medicine B is last administered 23rd 11:04 AM, then it should show the column upto 23rd 11:00 AM (because that column contains all administrations between 11 AM and 11:59 AM)

@print-Sathvik
Copy link
Contributor

End date: 23/09/2023

Sorry for taking an extra day. Before assigning this issue to myself I got the solution and thought that I will be completing it in few minutes but I misunderstood the issue earlier before your explanation. After trying out a different approach I noticed that the 1st administration is not being displayed in the table when the 1st administration was sent into second page of paginated table. I thought it was because of my work and I was making changes to my work but that bug is already existing at care.ohc.network. I fixed that too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers P2
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants