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

Alternative library to replace wkhtmltopdf #316

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

euniceremoquillo
Copy link
Contributor

This uses tcpdf as an alternative to wkhtmltopdf library. It uses a new tab when saving a pdf to give a preview of the pdf to use.
Still working on to most css styles configurations needed to provide the same browser output.

This is part of MON-13267

Signed-off-by: Eunice Remoquillo eremoquillo@itrsgroup.com

This commit uses tcpdf instead of wkhtmltopdf library.
Still missing updates for running scripts on tcpdf and
code snippets for logs.
This commit enables opening the form in new tab for pdf download.
Removed check patterns that replaces images and forms.
Initial update on script that captures the report elements
and should be sent to the controller using ajax. This way,
regenerating the content for a pdf file will not be needed
in downloading more accurate report data.
This commit logs encountered error instead of displaying it.
Added id and name reference for div to be able to use identify html parts on scripts.
This commit captures the html strings using a function instead of script tags.
This commit sets the right variable name to be retrieved.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant