[PR #1755] [MERGED] feat: certificate qrcode #1829

New issue

Closed

opened 2026-02-26 20:31:15 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 20:31:15 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/documenso/documenso/pull/1755
Author: @ephraimduncan
Created: 4/18/2025
Status: ✅ Merged
Merged: 4/28/2025
Merged by: @Mythie

Base: main ← Head: feat/document-qrcode-rr7

📝 Commits (2)

609a7f1 feat: certificate qrcode
f19a030 fix: update qr implementation

📊 Changes

26 files changed (+423 additions, -31 deletions)

View changed files

📝 apps/remix/app/components/dialogs/document-resend-dialog.tsx (+3 -5)
➕ apps/remix/app/components/general/document/document-certificate-qr-view.tsx (+106 -0)
➕ apps/remix/app/components/general/share-document-download-button.tsx (+53 -0)
📝 apps/remix/app/components/tables/documents-table-action-button.tsx (+2 -6)
📝 apps/remix/app/components/tables/documents-table-action-dropdown.tsx (+2 -6)
📝 apps/remix/app/components/tables/documents-table-title.tsx (+2 -6)
📝 apps/remix/app/routes/_internal+/[__htmltopdf]+/certificate.tsx (+14 -1)
➕ apps/remix/app/routes/_share+/_layout.tsx (+11 -0)
📝 apps/remix/app/routes/_share+/share.$slug.opengraph.tsx (+5 -0)
📝 apps/remix/app/routes/_share+/share.$slug.tsx (+37 -3)
📝 packages/lib/jobs/definitions/internal/seal-document.handler.ts (+12 -0)
📝 packages/lib/server-only/document/create-document-v2.ts (+2 -1)
📝 packages/lib/server-only/document/create-document.ts (+2 -0)
📝 packages/lib/server-only/document/duplicate-document-by-id.ts (+2 -0)
➕ packages/lib/server-only/document/get-document-by-access-token.ts (+38 -0)
📝 packages/lib/server-only/template/create-document-from-direct-template.ts (+2 -1)
📝 packages/lib/server-only/template/create-document-from-template-legacy.ts (+2 -1)
📝 packages/lib/server-only/template/create-document-from-template.ts (+2 -1)
📝 packages/lib/types/document.ts (+4 -0)
📝 packages/lib/universal/id.ts (+6 -0)

...and 6 more files

📄 Description

Demo

https://github.com/user-attachments/assets/66a1e744-e924-4c79-bbaf-8e6ace7f894a

Description

Adds document access tokens and QR code functionality to enable secure document sharing via URLs. It includes a new document access page that allows viewing and downloading documents through tokenized links.

Changes Made

Added new DocumentAccessToken model and migration to support secure document access
Created new document access page at /q/[token] for viewing shared documents
Added QR code generation to signing certificates using uqr library

Checklist

I have tested these changes locally and they work as expected
I have added appropriate database migrations
I have updated the documentation to reflect the new sharing functionality
I have followed the project's coding style guidelines
I have implemented proper error handling and security measures
I have tested the changes across different browsers and devices

_{🔄 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/1755 **Author:** [@ephraimduncan](https://github.com/ephraimduncan) **Created:** 4/18/2025 **Status:** ✅ Merged **Merged:** 4/28/2025 **Merged by:** [@Mythie](https://github.com/Mythie) **Base:** `main` ← **Head:** `feat/document-qrcode-rr7` --- ### 📝 Commits (2) - [`609a7f1`](https://github.com/documenso/documenso/commit/609a7f13daedc4ab8f73f155043ac8eee4935729) feat: certificate qrcode - [`f19a030`](https://github.com/documenso/documenso/commit/f19a030c6bf6be5c7382cb271731f997f79a0232) fix: update qr implementation ### 📊 Changes **26 files changed** (+423 additions, -31 deletions) <details> <summary>View changed files</summary> 📝 `apps/remix/app/components/dialogs/document-resend-dialog.tsx` (+3 -5) ➕ `apps/remix/app/components/general/document/document-certificate-qr-view.tsx` (+106 -0) ➕ `apps/remix/app/components/general/share-document-download-button.tsx` (+53 -0) 📝 `apps/remix/app/components/tables/documents-table-action-button.tsx` (+2 -6) 📝 `apps/remix/app/components/tables/documents-table-action-dropdown.tsx` (+2 -6) 📝 `apps/remix/app/components/tables/documents-table-title.tsx` (+2 -6) 📝 `apps/remix/app/routes/_internal+/[__htmltopdf]+/certificate.tsx` (+14 -1) ➕ `apps/remix/app/routes/_share+/_layout.tsx` (+11 -0) 📝 `apps/remix/app/routes/_share+/share.$slug.opengraph.tsx` (+5 -0) 📝 `apps/remix/app/routes/_share+/share.$slug.tsx` (+37 -3) 📝 `packages/lib/jobs/definitions/internal/seal-document.handler.ts` (+12 -0) 📝 `packages/lib/server-only/document/create-document-v2.ts` (+2 -1) 📝 `packages/lib/server-only/document/create-document.ts` (+2 -0) 📝 `packages/lib/server-only/document/duplicate-document-by-id.ts` (+2 -0) ➕ `packages/lib/server-only/document/get-document-by-access-token.ts` (+38 -0) 📝 `packages/lib/server-only/template/create-document-from-direct-template.ts` (+2 -1) 📝 `packages/lib/server-only/template/create-document-from-template-legacy.ts` (+2 -1) 📝 `packages/lib/server-only/template/create-document-from-template.ts` (+2 -1) 📝 `packages/lib/types/document.ts` (+4 -0) 📝 `packages/lib/universal/id.ts` (+6 -0) _...and 6 more files_ </details> ### 📄 Description ## Demo https://github.com/user-attachments/assets/66a1e744-e924-4c79-bbaf-8e6ace7f894a ## Description Adds document access tokens and QR code functionality to enable secure document sharing via URLs. It includes a new document access page that allows viewing and downloading documents through tokenized links. ## Changes Made - Added new `DocumentAccessToken` model and migration to support secure document access - Created new document access page at `/q/[token]` for viewing shared documents - Added QR code generation to signing certificates using `uqr` library ## Checklist - [x] I have tested these changes locally and they work as expected - [x] I have added appropriate database migrations - [x] I have updated the documentation to reflect the new sharing functionality - [x] I have followed the project's coding style guidelines - [x] I have implemented proper error handling and security measures - [x] I have tested the changes across different browsers and devices --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-02-26 20:31:15 +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#1829

No description provided.

Rows
Columns