-
-
Notifications
You must be signed in to change notification settings - Fork 2
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(routes): add html-to-pdf route #1440
base: main
Are you sure you want to change the base?
Conversation
Update index.js
New Feature - HTML TO PDF
@NHS-juju could you enable Allow edits from maintainers on this PR, please? |
@Fdawgs I can't see that option, and the page you linked suggests it is because the fork is from an Organisation account and the feature is only available when the fork is owned by a personal account 😐 I attempted to resolve the conflicts via the browser, and now merging is blocked... my bad! |
Known issues: html-to-pdf plugin lacks proper input validation, so currently will render anything a browser would attempt to render and prints this to a PDF file. In turn, the testing suites for this plugin are lacking in quality compared to the overall project.