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

fix: document metadata setters #2949

Merged
merged 1 commit into from
Nov 16, 2024
Merged

fix: document metadata setters #2949

merged 1 commit into from
Nov 16, 2024

Conversation

diegomura
Copy link
Owner

Closes: #2813

Existing solution works well, but PDF constructor sets /ID deterministically based on the document info, and setting that at rendering time is too late.

@diegomura diegomura merged commit fb848a7 into master Nov 16, 2024
@diegomura diegomura deleted the dm/fix-metadata-setters branch November 16, 2024 23:27
diegomura added a commit that referenced this pull request Nov 17, 2024
natterstefan added a commit to traveltechdeluxe/react-pdf that referenced this pull request Dec 6, 2024
# By Diego Muracciole (31) and others
# Via GitHub
* upstream/master: (60 commits)
  feat: allow units for page size (diegomura#2773)
  chore: bump jay-peg
  chore: release packages (diegomura#2981)
  fix(textkit): make indentation only affect first line. (diegomura#2975)
  fix: conditional rendering (diegomura#2983)
  fix(reconciler): missing dependencies (diegomura#2980)
  chore: release packages (diegomura#2959)
  feat: rem border widths (diegomura#2960)
  fix: add scheduler dependency (diegomura#2958)
  chore: update README
  chore: release packages (diegomura#2953)
  feat: support rem units (diegomura#2955)
  feat: add image stress test example (diegomura#2954)
  feat: support multiple line-height units (diegomura#2952)
  chore: release packages (diegomura#2946)
  fix: note rendering (diegomura#2951)
  feat: accept commas between transformations (diegomura#2950)
  fix: document metadata setters (diegomura#2949)
  fix: stroke dash array computation (diegomura#2948)
  feat: remove cross-fetch (diegomura#2947)
  ...

# Conflicts:
#	packages/renderer/index.d.ts
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.

Non-deterministic Document IDs on subsequent renders
1 participant