[PR #1521] [MERGED] fix: document visibility logic #1689

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/documenso/documenso/pull/1521
Author: @catalinpit
Created: 12/9/2024
Status: ✅ Merged
Merged: 12/16/2024
Merged by: @catalinpit

Base: main ← Head: fix/document-visibility-logic

📝 Commits (9)

df31738 fix: document visibility logic
aaa2507 fix: document visibility be and fe logic
505baa7 fix: document visibility backend logic
a2d84bd fix: document visibility backend logic again
30d2d33 chore: add tests
6821536 chore: add tests and update existing ones
4fa3cb4 Merge branch 'main' into fix/document-visibility-logic
ae6271b chore: update the documentation for doc visibility
db7f581 Merge branch 'main' into fix/document-visibility-logic

📊 Changes

7 files changed (+303 additions, -57 deletions)

View changed files

📝 apps/documentation/pages/users/teams/document-visibility.mdx (+11 -9)
📝 packages/app-tests/e2e/teams/team-documents.spec.ts (+225 -4)
📝 packages/lib/server-only/document/create-document.ts (+5 -6)
📝 packages/lib/server-only/document/update-document-settings.ts (+32 -28)
📝 packages/prisma/seed/teams.ts (+5 -0)
📝 packages/ui/components/document/document-visibility-select.tsx (+23 -9)
📝 packages/ui/primitives/document-flow/add-settings.tsx (+2 -1)

📄 Description

Update the logic of document visibility logic and added some tests & updated some existing ones.

_{🔄 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/1521 **Author:** [@catalinpit](https://github.com/catalinpit) **Created:** 12/9/2024 **Status:** ✅ Merged **Merged:** 12/16/2024 **Merged by:** [@catalinpit](https://github.com/catalinpit) **Base:** `main` ← **Head:** `fix/document-visibility-logic` --- ### 📝 Commits (9) - [`df31738`](https://github.com/documenso/documenso/commit/df31738c938c37ea66bce0df01bdabd4b31f127b) fix: document visibility logic - [`aaa2507`](https://github.com/documenso/documenso/commit/aaa250722f85a59118d033e3cb4721277761ab29) fix: document visibility be and fe logic - [`505baa7`](https://github.com/documenso/documenso/commit/505baa7537221c142780d376e93eaf20ba2f4ef4) fix: document visibility backend logic - [`a2d84bd`](https://github.com/documenso/documenso/commit/a2d84bd12d4b85842ad1559c962046f388e34420) fix: document visibility backend logic again - [`30d2d33`](https://github.com/documenso/documenso/commit/30d2d3361496917c3222502fb77b5da037654a03) chore: add tests - [`6821536`](https://github.com/documenso/documenso/commit/6821536a49e6c229123afad0a8e98b52dad038a6) chore: add tests and update existing ones - [`4fa3cb4`](https://github.com/documenso/documenso/commit/4fa3cb4d20293f34a57ffaa4a35022a2b3c0c21f) Merge branch 'main' into fix/document-visibility-logic - [`ae6271b`](https://github.com/documenso/documenso/commit/ae6271b52a2ab48bb88e942a0e0d4765a5b08368) chore: update the documentation for doc visibility - [`db7f581`](https://github.com/documenso/documenso/commit/db7f5814f4d5002bea0c26313365ffc4fabd489b) Merge branch 'main' into fix/document-visibility-logic ### 📊 Changes **7 files changed** (+303 additions, -57 deletions) <details> <summary>View changed files</summary> 📝 `apps/documentation/pages/users/teams/document-visibility.mdx` (+11 -9) 📝 `packages/app-tests/e2e/teams/team-documents.spec.ts` (+225 -4) 📝 `packages/lib/server-only/document/create-document.ts` (+5 -6) 📝 `packages/lib/server-only/document/update-document-settings.ts` (+32 -28) 📝 `packages/prisma/seed/teams.ts` (+5 -0) 📝 `packages/ui/components/document/document-visibility-select.tsx` (+23 -9) 📝 `packages/ui/primitives/document-flow/add-settings.tsx` (+2 -1) </details> ### 📄 Description Update the logic of document visibility logic and added some tests & updated some existing ones. --- <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:38 +03:00

closed this issue
added the
pull-request
label

kerem referenced this issue

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

[PR #1689] [MERGED] fix: prefill fields #1795

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

No description provided.

Rows
Columns