[PR #2251] [MERGED] fix: migrate certificate generation #2154

New issue

Closed

opened 2026-02-26 20:32:40 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 20:32:40 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/documenso/documenso/pull/2251
Author: @dguyen
Created: 11/27/2025
Status: ✅ Merged
Merged: 1/6/2026
Merged by: @Mythie

Base: main ← Head: fix/migrate-certificate-generation

📝 Commits (9)

411ff85 fix: migrate certificate generation
2d16bdd fix: increase cert size
7f7e943 fix: issues
b26ccdf Merge branch 'main' into fix/migrate-certificate-generation
f6b3fb3 Merge branch 'main' into fix/migrate-certificate-generation
f04355c fix: port fallback for certficate sent date when using link distribution
19d1b27 fix: memory leaks
707d954 Merge branch 'main' into fix/migrate-certificate-generation
8735ff5 feat: add env toggle for playwright-based pdf generation

📊 Changes

36 files changed (+2008 additions, -233 deletions)

View changed files

📝 apps/remix/app/components/forms/token.tsx (+4 -4)
📝 apps/remix/app/components/general/document-signing/document-signing-mobile-widget.tsx (+9 -9)
📝 apps/remix/app/components/general/document/document-page-view-recent-activity.tsx (+22 -22)
📝 apps/remix/app/components/general/envelope-editor/envelope-editor-settings-dialog.tsx (+8 -8)
📝 apps/remix/app/components/tables/document-logs-table.tsx (+1 -1)
📝 apps/remix/app/components/tables/internal-audit-log-table.tsx (+11 -11)
📝 apps/remix/app/routes/_authenticated+/o.$orgUrl._index.tsx (+13 -13)
📝 apps/remix/app/routes/_authenticated+/o.$orgUrl.settings.branding.tsx (+1 -1)
📝 apps/remix/app/routes/_internal+/[__htmltopdf]+/audit-log.tsx (+2 -2)
📝 apps/remix/app/routes/_recipient+/sign.$token+/_index.tsx (+6 -6)
📝 apps/remix/app/routes/_recipient+/sign.$token+/rejected.tsx (+4 -4)
📝 apps/remix/app/routes/_recipient+/sign.$token+/waiting.tsx (+2 -2)
📝 packages/app-tests/visual-regression/blank-certificate.png (+0 -0)
📝 packages/email/templates/confirm-team-email.tsx (+1 -1)
📝 packages/email/templates/reset-password.tsx (+1 -1)
📝 packages/lib/constants/app.ts (+9 -0)
📝 packages/lib/constants/pdf.ts (+5 -0)
📝 packages/lib/jobs/definitions/internal/seal-document.handler.ts (+77 -83)
📝 packages/lib/server-only/htmltopdf/get-audit-logs-pdf.ts (+3 -0)
📝 packages/lib/server-only/htmltopdf/get-certificate-pdf.ts (+3 -0)

...and 16 more files

📄 Description

Description

Generate certificates and audit logs using Konva instead of browserless.

This should:

Reduce the changes of generations failing
Improve sealing speed

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/documenso/documenso/pull/2251 **Author:** [@dguyen](https://github.com/dguyen) **Created:** 11/27/2025 **Status:** ✅ Merged **Merged:** 1/6/2026 **Merged by:** [@Mythie](https://github.com/Mythie) **Base:** `main` ← **Head:** `fix/migrate-certificate-generation` --- ### 📝 Commits (9) - [`411ff85`](https://github.com/documenso/documenso/commit/411ff85d67dfe4e381bffaeaa45f2a1c29f56f9a) fix: migrate certificate generation - [`2d16bdd`](https://github.com/documenso/documenso/commit/2d16bddef1a47eef6aca758697ef96183bd81653) fix: increase cert size - [`7f7e943`](https://github.com/documenso/documenso/commit/7f7e943031abf6536573009f3b27a9bf5656c463) fix: issues - [`b26ccdf`](https://github.com/documenso/documenso/commit/b26ccdf8c3994415ad44cab85f62cf6920ee6cd8) Merge branch 'main' into fix/migrate-certificate-generation - [`f6b3fb3`](https://github.com/documenso/documenso/commit/f6b3fb37a196df2bea19014cf65cd99850981ac4) Merge branch 'main' into fix/migrate-certificate-generation - [`f04355c`](https://github.com/documenso/documenso/commit/f04355c9d55d333f4048c7c780915bfce064a113) fix: port fallback for certficate sent date when using link distribution - [`19d1b27`](https://github.com/documenso/documenso/commit/19d1b27fcbf0d947f809a028786efea513c28c0f) fix: memory leaks - [`707d954`](https://github.com/documenso/documenso/commit/707d954bc708cc4671f348c3b865f93e59296fa8) Merge branch 'main' into fix/migrate-certificate-generation - [`8735ff5`](https://github.com/documenso/documenso/commit/8735ff5a95bd79bdc63b4ed8ce313ad01e927802) feat: add env toggle for playwright-based pdf generation ### 📊 Changes **36 files changed** (+2008 additions, -233 deletions) <details> <summary>View changed files</summary> 📝 `apps/remix/app/components/forms/token.tsx` (+4 -4) 📝 `apps/remix/app/components/general/document-signing/document-signing-mobile-widget.tsx` (+9 -9) 📝 `apps/remix/app/components/general/document/document-page-view-recent-activity.tsx` (+22 -22) 📝 `apps/remix/app/components/general/envelope-editor/envelope-editor-settings-dialog.tsx` (+8 -8) 📝 `apps/remix/app/components/tables/document-logs-table.tsx` (+1 -1) 📝 `apps/remix/app/components/tables/internal-audit-log-table.tsx` (+11 -11) 📝 `apps/remix/app/routes/_authenticated+/o.$orgUrl._index.tsx` (+13 -13) 📝 `apps/remix/app/routes/_authenticated+/o.$orgUrl.settings.branding.tsx` (+1 -1) 📝 `apps/remix/app/routes/_internal+/[__htmltopdf]+/audit-log.tsx` (+2 -2) 📝 `apps/remix/app/routes/_recipient+/sign.$token+/_index.tsx` (+6 -6) 📝 `apps/remix/app/routes/_recipient+/sign.$token+/rejected.tsx` (+4 -4) 📝 `apps/remix/app/routes/_recipient+/sign.$token+/waiting.tsx` (+2 -2) 📝 `packages/app-tests/visual-regression/blank-certificate.png` (+0 -0) 📝 `packages/email/templates/confirm-team-email.tsx` (+1 -1) 📝 `packages/email/templates/reset-password.tsx` (+1 -1) 📝 `packages/lib/constants/app.ts` (+9 -0) 📝 `packages/lib/constants/pdf.ts` (+5 -0) 📝 `packages/lib/jobs/definitions/internal/seal-document.handler.ts` (+77 -83) 📝 `packages/lib/server-only/htmltopdf/get-audit-logs-pdf.ts` (+3 -0) 📝 `packages/lib/server-only/htmltopdf/get-certificate-pdf.ts` (+3 -0) _...and 16 more files_ </details> ### 📄 Description ## Description Generate certificates and audit logs using Konva instead of browserless. This should: - Reduce the changes of generations failing - Improve sealing speed --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-02-26 20:32:40 +03:00

closed this issue
added the
pull-request
label

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/documenso#2154

No description provided.

Rows
Columns