[PR #1569] [CLOSED] feat: qr code on signing certificate #1723

Closed
opened 2026-02-26 20:30:47 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/documenso/documenso/pull/1569
Author: @ephraimduncan
Created: 1/3/2025
Status: Closed

Base: archive/nextjsHead: feat/document-qrcode


📝 Commits (10+)

  • f82e71f feat: qr code
  • 1fe5cda Merge branch 'main' into feat/document-qrcode
  • 1b1cd0b Merge branch 'main' into feat/document-qrcode
  • 99b0b80 chore: use uqr
  • e367894 Merge branch 'main' into feat/document-qrcode
  • 04e5244 chore: update styling
  • edfb1f2 feat: add document access token model
  • d82a759 feat: add document view page
  • 6743a10 chore: use correct link for qrcode
  • 810d194 feat: return the necessary information for document preview and download

📊 Changes

12 files changed (+310 additions, -14 deletions)

View changed files

apps/web/public/static/icon.svg (+19 -0)
📝 apps/web/src/app/(internal)/%5F%5Fhtmltopdf/certificate/page.tsx (+24 -12)
apps/web/src/app/(share)/q/[token]/document-download-button.tsx (+44 -0)
apps/web/src/app/(share)/q/[token]/layout.tsx (+35 -0)
apps/web/src/app/(share)/q/[token]/page.tsx (+68 -0)
📝 package-lock.json (+2 -1)
📝 packages/lib/jobs/definitions/internal/seal-document.handler.ts (+15 -0)
📝 packages/lib/server-only/admin/get-entire-document.ts (+1 -0)
packages/lib/server-only/document/get-document-by-access-token.ts (+39 -0)
📝 packages/lib/server-only/document/seal-document.ts (+15 -0)
packages/prisma/migrations/20250130053604_document_access_token/migration.sql (+29 -0)
📝 packages/prisma/schema.prisma (+19 -1)

📄 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/1569 **Author:** [@ephraimduncan](https://github.com/ephraimduncan) **Created:** 1/3/2025 **Status:** ❌ Closed **Base:** `archive/nextjs` ← **Head:** `feat/document-qrcode` --- ### 📝 Commits (10+) - [`f82e71f`](https://github.com/documenso/documenso/commit/f82e71f11c52bb4059a2c0343466344913767b8b) feat: qr code - [`1fe5cda`](https://github.com/documenso/documenso/commit/1fe5cda0ed261cb359ce435409f4af6ba25cc9c3) Merge branch 'main' into feat/document-qrcode - [`1b1cd0b`](https://github.com/documenso/documenso/commit/1b1cd0ba3df4f28c3bcf2dce67d6158714976ede) Merge branch 'main' into feat/document-qrcode - [`99b0b80`](https://github.com/documenso/documenso/commit/99b0b80635f012515a6a7dde7b2a521fd6e21c33) chore: use uqr - [`e367894`](https://github.com/documenso/documenso/commit/e367894218212c351b51fdba9b6495ce171bc00c) Merge branch 'main' into feat/document-qrcode - [`04e5244`](https://github.com/documenso/documenso/commit/04e5244a5a53fd5141bd6fda01dd08ff1f0a9150) chore: update styling - [`edfb1f2`](https://github.com/documenso/documenso/commit/edfb1f21572c7ac955eb2ba9d154f0d6bf37d074) feat: add document access token model - [`d82a759`](https://github.com/documenso/documenso/commit/d82a759acdb0f4c8f0ccbefa9729047bda17129c) feat: add document view page - [`6743a10`](https://github.com/documenso/documenso/commit/6743a108c0c5b5ffa1e63c167cb1723b5d6df935) chore: use correct link for qrcode - [`810d194`](https://github.com/documenso/documenso/commit/810d194cfd3f5e227e818d13c58322f21c342027) feat: return the necessary information for document preview and download ### 📊 Changes **12 files changed** (+310 additions, -14 deletions) <details> <summary>View changed files</summary> ➕ `apps/web/public/static/icon.svg` (+19 -0) 📝 `apps/web/src/app/(internal)/%5F%5Fhtmltopdf/certificate/page.tsx` (+24 -12) ➕ `apps/web/src/app/(share)/q/[token]/document-download-button.tsx` (+44 -0) ➕ `apps/web/src/app/(share)/q/[token]/layout.tsx` (+35 -0) ➕ `apps/web/src/app/(share)/q/[token]/page.tsx` (+68 -0) 📝 `package-lock.json` (+2 -1) 📝 `packages/lib/jobs/definitions/internal/seal-document.handler.ts` (+15 -0) 📝 `packages/lib/server-only/admin/get-entire-document.ts` (+1 -0) ➕ `packages/lib/server-only/document/get-document-by-access-token.ts` (+39 -0) 📝 `packages/lib/server-only/document/seal-document.ts` (+15 -0) ➕ `packages/prisma/migrations/20250130053604_document_access_token/migration.sql` (+29 -0) 📝 `packages/prisma/schema.prisma` (+19 -1) </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:30:47 +03:00
Sign in to join this conversation.
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#1723
No description provided.