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.

Sign up for GitHub

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

Jump to bottom

Bug: Exported XLS files has mismatched Patient's Age (Sample Test Management) #9231

Open
pooranjoyb opened this issue Nov 28, 2024 · 1 comment · May be fixed by #9247
Open

Bug: Exported XLS files has mismatched Patient's Age (Sample Test Management) #9231

pooranjoyb opened this issue Nov 28, 2024 · 1 comment · May be fixed by #9247
Assignees
@pooranjoyb
Labels
work-in-progress

Comments

@pooranjoyb
Copy link

pooranjoyb commented Nov 28, 2024 •
edited
Loading

Describe the bug
While exploring the features I came across this bug. Like when we export the Samples Data into excel sheet, everything seems fine, but some of the Patient's Age are not calculated in properly and hence parsed in with some random date in the excel cells.

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'Sample Management System'
  2. Click on 'Export Button'
  3. Open the XLS sheet
  4. Have a look on the Patient's Age column.
  5. Some Ages are not parsed properly.

Expected behavior
The Patient Ages should be properly populated accordingly.

Screenshots
Image

Desktop (please complete the following information):

  • OS: Linux
  • Browser: Chrome, Edge

Proposed Solution

  • For some records (could be some XLS parsing issue) the Patient's Age is interpreted as Date but not as the string.
  • We need to handle them manually such that if they are not parsed as Date but as strings.

care_fe/src/components/Patient/SampleViewAdmin.tsx

Lines 125 to 131 in fc2ac3b

row
.trim()
.split(",")
.map((field: string) =>
new Date(field).toString() === "Invalid Date"
? field
: formatDateTime(field),

Have a look onto this and lemme know if I can work on fixing this issue. @Jacobjeevan

cc @nihal467 @rithviknishad
@pooranjoyb
Copy link
Author

Any update on this? @Jacobjeevan

@github-actions github-actions bot added needs-triage question Further information is requested labels Nov 29, 2024
@Jacobjeevan Jacobjeevan removed question Further information is requested needs-triage labels Nov 29, 2024
@Jacobjeevan Jacobjeevan added this to Care Nov 29, 2024
@github-project-automation github-project-automation bot moved this to Triage in Care Nov 29, 2024
@Jacobjeevan Jacobjeevan moved this from Triage to Up Next in Care Nov 29, 2024
@pooranjoyb pooranjoyb linked a pull request Nov 29, 2024 that will close this issue
Open
6 tasks
@Jacobjeevan Jacobjeevan moved this from Up Next to Review required in Care Dec 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pooranjoyb pooranjoyb

Labels
work-in-progress
Projects
Care
Status: Review required
Milestone
No milestone
2 participants
@Jacobjeevan @pooranjoyb