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

Move gift cards PDF to react-pdf #983

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Move gift cards PDF to react-pdf #983

wants to merge 1 commit into from

Conversation

Betree
Copy link
Member

@Betree Betree commented Jan 8, 2024

@Betree Betree self-assigned this Jan 8, 2024
Copy link

vercel bot commented Jan 8, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
opencollective-pdf ✅ Ready (Inspect) Visit Preview Jan 8, 2024 2:02pm

// res.setHeader('Content-Disposition', 'attachment; filename="gift-cards.pdf"');

// Generate doc
const paginatedCards = chunk(cards, 8);

Check failure

Code scanning / CodeQL

Loop bound injection High

Iteration over a user-controlled object with a potentially unbounded .length property from a
user-provided value
.
@Betree Betree force-pushed the main branch 7 times, most recently from 32827ed to 999eec1 Compare February 19, 2024 15:17
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.

1 participant