starred/documenso

Fork 0

mirror of https://github.com/documenso/documenso.git synced 2026-04-25 13:55:59 +03:00

[PR #1566] [MERGED] fix: add document visibility to template #1721

New issue

Closed

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

kerem commented

2026-02-26 20:30:47 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/documenso/documenso/pull/1566
Author: @catalinpit
Created: 12/31/2024
Status: ✅ Merged
Merged: 1/8/2025
Merged by: @Mythie

Base: main ← Head: fix/team-template-visibility

📝 Commits (4)

eb9b04f fix: add document visibility to template
4e58e5e Merge branch 'main' into fix/team-template-visibility
d53df3f fix: add tests and update missing parts
2f5eba8 fix: update tests

📊 Changes

12 files changed (+316 additions, -26 deletions)

View changed files

📝 apps/web/src/app/(dashboard)/templates/[id]/edit/edit-template.tsx (+2 -0)
📝 packages/app-tests/e2e/templates-flow/template-settings-step.spec.ts (+108 -0)
📝 packages/app-tests/e2e/templates/create-document-from-template.spec.ts (+88 -1)
📝 packages/app-tests/e2e/templates/direct-templates.spec.ts (+2 -0)
📝 packages/lib/server-only/template/create-document-from-template.ts (+1 -1)
📝 packages/lib/server-only/template/find-templates.ts (+56 -17)
📝 packages/lib/server-only/template/update-template-settings.ts (+3 -1)
➕ packages/prisma/migrations/20241231121013_add_visibility_to_template/migration.sql (+2 -0)
📝 packages/prisma/schema.prisma (+7 -6)
📝 packages/trpc/server/template-router/schema.ts (+2 -0)
📝 packages/ui/primitives/template-flow/add-template-settings.tsx (+43 -0)
📝 packages/ui/primitives/template-flow/add-template-settings.types.tsx (+2 -0)

📄 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/1566 **Author:** [@catalinpit](https://github.com/catalinpit) **Created:** 12/31/2024 **Status:** ✅ Merged **Merged:** 1/8/2025 **Merged by:** [@Mythie](https://github.com/Mythie) **Base:** `main` ← **Head:** `fix/team-template-visibility` --- ### 📝 Commits (4) - [`eb9b04f`](https://github.com/documenso/documenso/commit/eb9b04feb7c566fa201687d66b19200ded09df22) fix: add document visibility to template - [`4e58e5e`](https://github.com/documenso/documenso/commit/4e58e5e2236e1b6c6ab34b39fe15ac2baac097ff) Merge branch 'main' into fix/team-template-visibility - [`d53df3f`](https://github.com/documenso/documenso/commit/d53df3f9fe2115ae8caa960a0a101ea47ef5a7c6) fix: add tests and update missing parts - [`2f5eba8`](https://github.com/documenso/documenso/commit/2f5eba8a3b1174dd92bed85a02c4ae9f8e830a56) fix: update tests ### 📊 Changes **12 files changed** (+316 additions, -26 deletions) <details> <summary>View changed files</summary> 📝 `apps/web/src/app/(dashboard)/templates/[id]/edit/edit-template.tsx` (+2 -0) 📝 `packages/app-tests/e2e/templates-flow/template-settings-step.spec.ts` (+108 -0) 📝 `packages/app-tests/e2e/templates/create-document-from-template.spec.ts` (+88 -1) 📝 `packages/app-tests/e2e/templates/direct-templates.spec.ts` (+2 -0) 📝 `packages/lib/server-only/template/create-document-from-template.ts` (+1 -1) 📝 `packages/lib/server-only/template/find-templates.ts` (+56 -17) 📝 `packages/lib/server-only/template/update-template-settings.ts` (+3 -1) ➕ `packages/prisma/migrations/20241231121013_add_visibility_to_template/migration.sql` (+2 -0) 📝 `packages/prisma/schema.prisma` (+7 -6) 📝 `packages/trpc/server/template-router/schema.ts` (+2 -0) 📝 `packages/ui/primitives/template-flow/add-template-settings.tsx` (+43 -0) 📝 `packages/ui/primitives/template-flow/add-template-settings.types.tsx` (+2 -0) </details> ### 📄 Description ## Description   ## Related Issue   ## 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   --- <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

closed this issue
added the
pull-request
label

kerem referenced this issue

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

[PR #1721] [MERGED] fix: duplicate webhook calls on document complete #1812

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#1721

No description provided.

Rows
Columns