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 Report: Copy to Clipboard Function Showing Age as NaN on Shifting Details Page #8981

Closed
Rishith25 opened this issue Nov 1, 2024 · 6 comments · Fixed by #8984
Closed
Assignees
Labels
good first issue Good for newcomers

Comments

@Rishith25
Copy link
Contributor

Issue Description: On the "Shifting Details" page, the "Copy to Clipboard" function is not copying the "Age" field correctly. Instead of showing the actual age, it displays "NaN."

Steps to Reproduce:

Navigate to the "Shifting Details" page.
Locate the details for a specific person (e.g., Aditya Jindal).
Click the "Copy to Clipboard" button to copy all details.
Paste the copied text into a text editor or any other application to view the output.
Observed Result: The copied text shows "Age: NaN" instead of the actual age value.

Expected Result: The "Age" field should display the correct numerical age when copied to the clipboard.

Additional Information: Could you let us know if any other details should be included in the "Copy to Clipboard" function? For example, it might be helpful to add an "Emergency Status" field to clarify if this is an urgent or non-urgent shift. Please let us know if there are other details that should also be included in the copied information.

Name:Aditya Jindal
Age:NaN
Current facility:Aloha New
Contact Number:+917719415700
Address:House No. 172 Balvir Singh Street
Reason:Contrary to popular belief, Lorem Ipsum is not simply .......
@Rishith25
Copy link
Contributor Author

@rithviknishad I would like to work on this

@AdityaJ2305
Copy link
Contributor

Hey @rithviknishad, how can I delete the patients created under my name?

@Rishith25
Copy link
Contributor Author

@AdityaJ2305 Hey there this is related to another issue

@mnshrm
Copy link
Contributor

mnshrm commented Nov 2, 2024

Hi @rithviknishad,
I found solution to this issue, can raise PR in under 10 minutes for this issue, please assign the issue to me

Regards

@rithviknishad
Copy link
Member

@Rishith25

Hey there this is related to another issue

Could you reference the related issue in such situations?

@mnshrm
Copy link
Contributor

mnshrm commented Nov 2, 2024

Hi @rithviknishad,
I have raised PR and linked this issue, Please let me know if any discrepancies are found, Happy to contribute to OHN

Regards

@github-project-automation github-project-automation bot moved this from Up Next to Done in Care Nov 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants