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

feat: Print Merged lab test report from list view #27642

Conversation

mujeerhashmi
Copy link
Contributor

@mujeerhashmi mujeerhashmi commented Sep 23, 2021

print_merged

report

Kindly check the demo

print_merged_lab_test.mp4

@ChillarAnand
Copy link
Contributor

@mujeerhashmi Thanks for raising the PR.

Build is failing due to lint issues. Can you please fix that?

Signed-off-by: Syed Mujeer Hashmi <mujeerhashmi@4csolutions.in>
@mujeerhashmi
Copy link
Contributor Author

@mujeerhashmi Thanks for raising the PR.

Build is failing due to lint issues. Can you please fix that?

OK Done. I have fixed SIder and Linter Issues.

@ChillarAnand
Copy link
Contributor

Currently, user can select multiple items and click on print. It will print selected items in multiple documents.

To merge these documents and print them, we can may be add an option in the framework itself so that multiple docs can be merged and printed?
We might need these merging for other doctypes also.

Any suggestions? @surajshetty3416 @leela

@stale
Copy link

stale bot commented Oct 15, 2021

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed within a week if no further activity occurs, but it only takes a comment to keep a contribution alive :) Also, even if it is closed, you can always reopen the PR when you're ready. Thank you for contributing.

@stale stale bot added the inactive label Oct 15, 2021
@mujeerhashmi
Copy link
Contributor Author

Keep alive

@stale stale bot removed the inactive label Oct 15, 2021
@stale
Copy link

stale bot commented Oct 30, 2021

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed within a week if no further activity occurs, but it only takes a comment to keep a contribution alive :) Also, even if it is closed, you can always reopen the PR when you're ready. Thank you for contributing.

@stale stale bot added the inactive label Oct 30, 2021
@mujeerhashmi
Copy link
Contributor Author

Keep Alive

@stale stale bot removed the inactive label Oct 30, 2021
@stale
Copy link

stale bot commented Nov 14, 2021

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed within a week if no further activity occurs, but it only takes a comment to keep a contribution alive :) Also, even if it is closed, you can always reopen the PR when you're ready. Thank you for contributing.

@stale stale bot added the inactive label Nov 14, 2021
@stale stale bot closed this Nov 17, 2021
@tribikramsahu
Copy link

This feature is most needed!

@ruchamahabal
Copy link
Member

@tribikramsahu You can use the standard framework feature for bulk printing that the framework provides:

image

image

@tribikramsahu
Copy link

@tribikramsahu You can use the standard framework feature for bulk printing that the framework provides:

image image

This creates a page break on each document end, I want the merged print without any page break or repeated header.

@mujeerhashmi
Copy link
Contributor Author

mujeerhashmi commented May 10, 2022

@ruchamahabal This feat is a must in any healthcare facility willing to save pages for printed reports. The original purpose was not for bulk printing but to merge reports that do not need a complete page.

I raised this PR because we could not have achieved this without custom app. This one is generic requirement in most of the labs.

@ChillarAnand
Copy link
Contributor

@mujeerhashmi Bulk printing is needed not only for labs, it is needed for other doctypes like encounters, patient history, etc. It is needed even in non-healthcare modules as well.

This creates a page break on each document end, I want the merged print without any page break or repeated header.

May be add a section for bulk printing in print settings and make it configurable? So that users can print with or without page breaks or repeated headers? @tribikramsahu

@tribikramsahu
Copy link

@ChillarAnand Yes, that would be good enough to proceed.

@aakvatech
Copy link
Contributor

Why is this closed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants