[PR #1104] [MERGED] fix: duplicate templates #1388

Closed
opened 2026-02-26 19:33:02 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/documenso/documenso/pull/1104
Author: @dguyen
Created: 4/15/2024
Status: Merged
Merged: 4/17/2024
Merged by: @dguyen

Base: mainHead: fix/duplicate-templates


📝 Commits (1)

📊 Changes

1 file changed (+62 additions, -55 deletions)

View changed files

📝 apps/web/src/app/(dashboard)/templates/new-template-dialog.tsx (+62 -55)

📄 Description

Description

Fix the issue where duplicate templates can be created

Changes Made

  • Prevent duplicate templates by correctly disabling elements
  • Clear the PDF when the form is reset
  • General UI changes to new template dialog for consistency
  • Add description for new template dialog
  • Add close button for new template dialog

Testing Performed

Manual testing

Checklist

  • I have tested these changes locally and they work as expected.
  • I have followed the project's coding style guidelines.

Summary by CodeRabbit

  • New Features

    • Introduced new components in the template creation dialog for better usability: DialogClose, DialogDescription, and DialogFooter.
    • Enhanced file upload handling and display.
    • Added a cancel button and a create template button to the dialog footer for improved navigation.
  • Refactor

    • Updated the dialog content and form layout for a more intuitive user experience.
    • Refactored form structure and labels for clarity.
  • Removed Features

    • Removed the Label component from the dialog.

🔄 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/1104 **Author:** [@dguyen](https://github.com/dguyen) **Created:** 4/15/2024 **Status:** ✅ Merged **Merged:** 4/17/2024 **Merged by:** [@dguyen](https://github.com/dguyen) **Base:** `main` ← **Head:** `fix/duplicate-templates` --- ### 📝 Commits (1) - [`2dd8c4c`](https://github.com/documenso/documenso/commit/2dd8c4c768de266a79beaa55df28520852fb50ec) fix: duplicate templates ### 📊 Changes **1 file changed** (+62 additions, -55 deletions) <details> <summary>View changed files</summary> 📝 `apps/web/src/app/(dashboard)/templates/new-template-dialog.tsx` (+62 -55) </details> ### 📄 Description ## Description Fix the issue where duplicate templates can be created ## Changes Made - Prevent duplicate templates by correctly disabling elements - Clear the PDF when the form is reset - General UI changes to new template dialog for consistency - Add description for new template dialog - Add close button for new template dialog ## Testing Performed Manual testing ## Checklist - [X] I have tested these changes locally and they work as expected. - [X] I have followed the project's coding style guidelines. <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Introduced new components in the template creation dialog for better usability: `DialogClose`, `DialogDescription`, and `DialogFooter`. - Enhanced file upload handling and display. - Added a cancel button and a create template button to the dialog footer for improved navigation. - **Refactor** - Updated the dialog content and form layout for a more intuitive user experience. - Refactored form structure and labels for clarity. - **Removed Features** - Removed the `Label` component from the dialog. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 19:33:02 +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#1388
No description provided.