Skip to content

Option to update existing signature box (instead of always creating a new one)#89

Closed
ridgey-dev wants to merge 2 commits intodealfonso:mainfrom
ridgey-dev:sign-document-existing-signature-box
Closed

Option to update existing signature box (instead of always creating a new one)#89
ridgey-dev wants to merge 2 commits intodealfonso:mainfrom
ridgey-dev:sign-document-existing-signature-box

Conversation

@ridgey-dev
Copy link
Contributor

@ridgey-dev ridgey-dev commented Jan 7, 2025

Currently, the package automatically creates a new signature box with a random name for every signature action. This behavior leads to PDFs that contain both the newly created (signed) signature box and the original (unsigned) placeholder, which is not ideal for workflows where maintaining the integrity of pre-existing placeholders is critical.

To address this, I have created this pull request to enable the functionality of updating existing signature boxes instead of creating new ones. This enhancement ensures that the package can effectively handle documents with predefined signature placeholders without introducing redundancies.

@ridgey-dev ridgey-dev changed the title Option to update existing signature box (instead of always creating a new one Option to update existing signature box (instead of always creating a new one) Jan 7, 2025
@angeljqv
Copy link

It looks feasible, but there seems to be no review

@parallels999
Copy link
Contributor

parallels999 commented Aug 6, 2025

What do you think about this idea?

If there are already placeholders, take the first empty one it finds.
It would save you the trouble of knowing the OID

Another idea,
a method that extracts the available placeholder IDs, so you can choose which one you're going to add the signature to.

@erikn69
Copy link
Contributor

erikn69 commented Aug 7, 2025

If you edit the object, a signature error would appear when adding the second signature in the second placeholder.

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.

4 participants

Comments