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

Bug: Generating a big Booking overview PDF breaks the whole app #1547

Closed
DonKoko opened this issue Dec 30, 2024 · 0 comments · Fixed by #1553
Closed

Bug: Generating a big Booking overview PDF breaks the whole app #1547

DonKoko opened this issue Dec 30, 2024 · 0 comments · Fixed by #1553
Assignees
Labels
bug Something isn't working Priority: High Issues that are with very high priority

Comments

@DonKoko
Copy link
Contributor

DonKoko commented Dec 30, 2024

When we generate a PDF overview of a big booking with many assets, it runs out of ram because we use headless chrome and it consumes all the server ram because chrome likes to munch on ram.

Solution:

Move this to be completely client side and use the browser's print functionality to create the PDF.
We can use the html we already have and change it to react so we have render the preview of the PDF before showing a download button.

@DonKoko DonKoko converted this from a draft issue Dec 30, 2024
@DonKoko DonKoko added bug Something isn't working Priority: High Issues that are with very high priority labels Dec 30, 2024
@rockingrohit9639 rockingrohit9639 moved this from 🗓️ Planned to 🏗 In progress in 🗺️ Shelf Roadmap Jan 2, 2025
@DonKoko DonKoko self-assigned this Jan 2, 2025
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in 🗺️ Shelf Roadmap Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Priority: High Issues that are with very high priority
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant