Skip to content

fix: reject oversized PDFs in reorder export - #631

Merged
Durgeshwar-AI merged 1 commit into
Durgeshwar-AI:mainfrom
nyxsky404:fix/607-block-oversized-reorder-export
Aug 10, 2026
Merged

fix: reject oversized PDFs in reorder export#631
Durgeshwar-AI merged 1 commit into
Durgeshwar-AI:mainfrom
nyxsky404:fix/607-block-oversized-reorder-export

Conversation

@nyxsky404

Copy link
Copy Markdown
Contributor

Pull Request

Issue Reference

Closes #607


Summary

  • reject reorder uploads over 50 pages instead of exporting a truncated PDF
  • keep export disabled when the page-limit warning is active
  • add regression coverage for truncated 51-page preview detection

Screenshots (if applicable)

NA


Checklist

  • My code follows the project's coding conventions
  • I have tested all impacted features
  • I have updated or added necessary documentation

Related Issues / PRs

NA


Open Source Program Participation

Program Name: GSSoC


Additional Notes

Focused Vitest checks, typecheck, and production build pass.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Aug 6, 2026

Copy link
Copy Markdown

@nyxsky404 is attempting to deploy a commit to the Durgeshwar's projects Team on Vercel.

A member of the Team first needs to authorize it.

@nyxsky404

Copy link
Copy Markdown
Contributor Author

@Durgeshwar-AI This PR is mergeable (no conflicts with main).

The failing backend CI (test_add_watermark_text_success / test_add_watermark_image_successValueError: not enough values to unpack in blueprints/watermark.py) is already present on upstream main, and is unrelated to this PR’s reorder size-limit changes. The Vercel check is failing on team authorization, not on this PR’s code.

@Durgeshwar-AI
Durgeshwar-AI merged commit 2f8d049 into Durgeshwar-AI:main Aug 10, 2026
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] PDF Reorder permits export after 50-page warning and drops later pages

2 participants