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

File Upload enhancement- multi-file support #7417

Open
aparnacoronasafe opened this issue Mar 15, 2024 · 15 comments
Open

File Upload enhancement- multi-file support #7417

aparnacoronasafe opened this issue Mar 15, 2024 · 15 comments
Assignees

Comments

@aparnacoronasafe
Copy link
Member

aparnacoronasafe commented Mar 15, 2024

Is your feature request related to a problem? Please describe.
Today file upload is used to upload relevant documents for the patient encounter, such as lab tests, scan reports, consent forms etc.
Many of such documents would consist of more than 1 page.

Currently, nurses are uploading each page as a separate file upload. like Blood test of 15/03/24 (page 1), Blood test of 15/03/24 (page 2) and so on.

This is not ideal for maintaining records, since continuity of the pages may be lost.

The users are not tech savvy enough or don't usually have the time to merge these different images into one PDF and upload (although that would have been the ideal thing to do)

Describe the solution you'd like
Allow multiple files to be added within 1 file upload.

@gigincg

@saloni0419
Copy link
Contributor

@aparnacoronasafe can you please assign this issue to me

@itxsoumya
Copy link
Contributor

@aparnacoronasafe I would like to take this issue.

First of all we have to create an option to upload multiple files in single click. For this we have to take all selected files and have to merge them together to a single PDF file and then upload that file to the server.

I already have worked with file upload functionality in care_fe and I think this would also be a great issue to solve. 🙌

@hrit2773
Copy link
Contributor

@aparnacoronasafe plz assign me this issue and tell me the credentials and under which section the issue is ?

@Thanush19
Copy link

hey @aparnacoronasafe , i wish to work on this....

@gigincg
Copy link
Member

gigincg commented Mar 20, 2024

Implementation Details:

What is required is a Scan and Upload Document Feature. We need to allow users to Scan multiple Images of a Document; Build a PDF out of it & Upload it as a single file.

Any packages that are required to enable this feature should be lazy loaded only when the Scan Document Feature is used.

Ref:

Recommended Package:
https://www.npmjs.com/package/jspdf

For using Rendering the Images in the PDF:
https://stackoverflow.com/questions/19065562/add-image-in-pdf-using-jspdf

NB: Feel free to suggest alternative implementations in this discussion thread if you have a better approach

@gigincg
Copy link
Member

gigincg commented Mar 20, 2024

@itxsoumya Your suggested solution seems inline with the required Implementation detail. Assigned the Issue to you. Please acknowledge here.

cc: @nihal467

@gigincg gigincg moved this from Triage to Up Next in Care Mar 20, 2024
@itxsoumya
Copy link
Contributor

@gigincg you have mentioned 'scan the documents and then merge them together '.

Does the scan refer to scanning the document using the camera or selecting the documents from the file manager?

@rithviknishad
Copy link
Member

How about having both options?

cc: @gigincg

@gigincg
Copy link
Member

gigincg commented Mar 22, 2024

Selecting Images or Taking snapshots

@itxsoumya @rithviknishad

@itxsoumya
Copy link
Contributor

@gigincg selecting images would be a little easier than taking snapshots and then processing

Copy link

github-actions bot commented Apr 6, 2024

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

@github-actions github-actions bot added the stale label Apr 6, 2024
@UdaySagar-Git
Copy link
Contributor

@itxsoumya Are you still working on this issue? If not, I would like to take it up.

@itxsoumya
Copy link
Contributor

@UdaySagar-Git Yup I am working

@github-actions github-actions bot removed the stale label Apr 7, 2024
@itxsoumya
Copy link
Contributor

@rithviknishad
Copy link
Member

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Up Next
Development

Successfully merging a pull request may close this issue.

8 participants