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(pdf): Allows creation of pdfs with custom dimensions #12

Merged
merged 1 commit into from
Aug 7, 2024

Conversation

FritzOnFire
Copy link
Collaborator

Moves a lot of code that is only needed by the png format path. This allows us to send the raw width and height to puppeteer for pdf's which allows callers to specify units with their widths and heights.

Refs: ASG-3281, ASG-3282

@FritzOnFire FritzOnFire added the enhancement New feature or request label Jul 4, 2024
@FritzOnFire FritzOnFire self-assigned this Jul 4, 2024
Copy link
Contributor

@anzelIMQS anzelIMQS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link

@DevinFledermaus DevinFledermaus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link

@CelesteNaude CelesteNaude left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@AshtonMar AshtonMar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AshtonMar AshtonMar force-pushed the asg-fz-custom-pdf-dimentions branch from 06e9455 to e6375cd Compare August 2, 2024 09:46
@AshtonMar AshtonMar merged commit b1b30ab into master Aug 7, 2024
1 check passed
@AshtonMar AshtonMar deleted the asg-fz-custom-pdf-dimentions branch August 7, 2024 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants