Skip to content

feat(esign): several places per signer and signing several documents with one PIN - #105

Merged
lorenzocorallo merged 4 commits into
mainfrom
feat/esign-places-batch
Oct 17, 2026
Merged

lorenzocorallo merged 4 commits into
mainfrom
feat/esign-places-batch

Conversation

@lorenzocorallo

@lorenzocorallo lorenzocorallo commented Oct 17, 2026 •

Copy link
Copy Markdown
Member

Dashboard side of PoliNetworkOrg/backend#75 and PoliNetworkOrg/esign#6.

Several places per signer, the person shown once

  • New request: under each signer, one button per place (Signature · p. 2, Mark · p. 1) and Add place (click on the page). A place's menu switches Full signature / Mark only, moves it or removes it, and rings its box on the PDF; the last full signature can't become a mark. Boxes of one person share their number; marks are dashed.
  • Request page: the signer drags all their boxes; signers list and activity show each person once ("1 signature and 1 mark"); a signer interrupted part way sees "1 of 2 signatures done".

Several documents with one PIN

  • To sign: Sign N together when more than one document waits; the request page suggests it too.
  • /dashboard/esign/sign: a strip of the documents (toolbar row); opening one marks it checked, each can be left out, and the primary action is Next document until all are checked, then Sign N documents.
  • Sign with CIE takes several documents: one popup with every ticket, per-document progress and errors, a shared error shown once (wrong PIN), partial results reported.

Checked: pnpm check, typecheck, test, build against the backend branch's built types. In the browser with the dev agent (software key only): created a request with a signature + a mark, signed it together with another document in one agent window → both completed, PDF valid with the mark drawn; a request with two full signatures → two valid PDF signatures, one signed event.

Backend#75 is released as @polinetwork/backend 1.9.0, used here. The upload fallback now warns signers with several places that it adds a single signature.

…with one PIN

A signer can sign a document in several places, each a full signature or a mark of the first one. The person
appears once everywhere: their places are buttons under their name, whose menu switches the kind, moves or removes
the place and rings its box on the page.

"Sign N together" in To sign opens the waiting documents in a strip to check one by one, then one agent window and
one PIN sign them all. The request page offers it when other documents wait, and Sign with CIE follows several jobs,
showing each document's result.

Needs the backend release with places and batch jobs (backend#75) and sign-agent v0.2.0 (sign-agent#6).
@coderabbitai

coderabbitai Bot commented Oct 17, 2026 •

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 46 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 494bdacd-380b-4fbe-90c7-2bcc5e337c37
📥 Commits

Reviewing files that changed from the base of the PR and between ec13d7f and 201a406.

⛔ Files ignored due to path filters: 1
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing: 17
  • docs/design.md
  • docs/esign/TRACKER.md
  • docs/esign/contracts.md
  • package.json
  • src/components/shell/nav.ts
  • src/features/esign/agent.ts
  • src/features/esign/batch-page.tsx
  • src/features/esign/esign.functions.ts
  • src/features/esign/new-request-page.tsx
  • src/features/esign/places.ts
  • src/features/esign/request-page.tsx
  • src/features/esign/requests-page.tsx
  • src/features/esign/requests.functions.ts
  • src/features/esign/sign-with-cie.tsx
  • src/features/esign/signature-box.tsx
  • src/routeTree.gen.ts
  • src/routes/dashboard/esign/sign.tsx
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

A signer with several places who uploads a PDF signed with the official app completes their part with one
signature, leaving the other places empty.
@lorenzocorallo
lorenzocorallo marked this pull request as ready for review October 17, 2026 10:37
# Conflicts:
#	docs/design.md
#	src/features/esign/sign-with-cie.tsx
@lorenzocorallo
lorenzocorallo merged commit fbe335b into main Oct 17, 2026
@lorenzocorallo
lorenzocorallo deleted the feat/esign-places-batch branch October 17, 2026 10:42
BIA3IA pushed a commit that referenced this pull request Oct 20, 2026
…with one PIN (#105)

* feat(esign): several places per signer and signing several documents with one PIN

A signer can sign a document in several places, each a full signature or a mark of the first one. The person
appears once everywhere: their places are buttons under their name, whose menu switches the kind, moves or removes
the place and rings its box on the page.

"Sign N together" in To sign opens the waiting documents in a strip to check one by one, then one agent window and
one PIN sign them all. The request page offers it when other documents wait, and Sign with CIE follows several jobs,
showing each document's result.

Needs the backend release with places and batch jobs (backend#75) and sign-agent v0.2.0 (sign-agent#6).

* feat(esign): warn that the upload fallback adds a single signature

A signer with several places who uploads a PDF signed with the official app completes their part with one
signature, leaving the other places empty.

* build(deps): use @polinetwork/backend 1.9.0 for places and batch jobs
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