[PR #2209] [MERGED] refactor: improve layout of completed signing page #2122

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

📋 Pull Request Information

Original PR: https://github.com/documenso/documenso/pull/2209
Author: @catalinpit
Created: 11/18/2025
Status: Merged
Merged: 11/20/2025
Merged by: @Mythie

Base: mainHead: fix/improve-complete-page


📝 Commits (3)

  • d4e259a refactor: improve layout of completed signing page
  • 0705228 feat: update completed signing page to redirect to folder when possible
  • 3dfee41 fix: excessive padding

📊 Changes

3 files changed (+28 additions, -12 deletions)

View changed files

📝 apps/remix/app/routes/_recipient+/sign.$token+/complete.tsx (+25 -10)
📝 packages/lib/server-only/document/get-document-by-token.ts (+1 -0)
📝 packages/ui/components/document/document-share-button.tsx (+2 -2)

📄 Description

Description

Changes Made

  • Change 1
  • Change 2
  • ...

Testing Performed

  • Tested feature X in scenario Y.
  • Ran unit tests for component Z.
  • Tested on browsers A, B, and C.
  • ...

Checklist

  • I have tested these changes locally and they work as expected.
  • I have added/updated tests that prove the effectiveness of these changes.
  • I have updated the documentation to reflect these changes, if applicable.
  • I have followed the project's coding style guidelines.
  • I have addressed the code review feedback from the previous submission, if applicable.

Additional Notes


🔄 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/2209 **Author:** [@catalinpit](https://github.com/catalinpit) **Created:** 11/18/2025 **Status:** ✅ Merged **Merged:** 11/20/2025 **Merged by:** [@Mythie](https://github.com/Mythie) **Base:** `main` ← **Head:** `fix/improve-complete-page` --- ### 📝 Commits (3) - [`d4e259a`](https://github.com/documenso/documenso/commit/d4e259a9a71a9c8c89f9f5aab06b8a2b92a240bf) refactor: improve layout of completed signing page - [`0705228`](https://github.com/documenso/documenso/commit/07052286096de82ae97be4d9cb2e7dde96fcc9ec) feat: update completed signing page to redirect to folder when possible - [`3dfee41`](https://github.com/documenso/documenso/commit/3dfee41d0e095dea926e9e6aa2ada926bad14052) fix: excessive padding ### 📊 Changes **3 files changed** (+28 additions, -12 deletions) <details> <summary>View changed files</summary> 📝 `apps/remix/app/routes/_recipient+/sign.$token+/complete.tsx` (+25 -10) 📝 `packages/lib/server-only/document/get-document-by-token.ts` (+1 -0) 📝 `packages/ui/components/document/document-share-button.tsx` (+2 -2) </details> ### 📄 Description ## Description <!--- Describe the changes introduced by this pull request. --> <!--- Explain what problem it solves or what feature/fix it adds. --> ## Related Issue <!--- If this pull request is related to a specific issue, reference it here using #issue_number. --> <!--- For example, "Fixes #123" or "Addresses #456". --> ## Changes Made <!--- Provide a summary of the changes made in this pull request. --> <!--- Include any relevant technical details or architecture changes. --> - Change 1 - Change 2 - ... ## Testing Performed <!--- Describe the testing that you have performed to validate these changes. --> <!--- Include information about test cases, testing environments, and results. --> - Tested feature X in scenario Y. - Ran unit tests for component Z. - Tested on browsers A, B, and C. - ... ## Checklist <!--- Please check the boxes that apply to this pull request. --> <!--- You can add or remove items as needed. --> - [ ] I have tested these changes locally and they work as expected. - [ ] I have added/updated tests that prove the effectiveness of these changes. - [ ] I have updated the documentation to reflect these changes, if applicable. - [ ] I have followed the project's coding style guidelines. - [ ] I have addressed the code review feedback from the previous submission, if applicable. ## Additional Notes <!--- Provide any additional context or notes for the reviewers. --> <!--- This might include details about design decisions, potential concerns, or anything else relevant. --> --- <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:31 +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#2122
No description provided.