[PR #1770] [MERGED] refactor: remove unused navigation and team context in folder dialogs #1836

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

📋 Pull Request Information

Original PR: https://github.com/documenso/documenso/pull/1770
Author: @catalinpit
Created: 5/2/2025
Status: Merged
Merged: 6/19/2025
Merged by: @dguyen

Base: mainHead: fix/disable-folder-navigation-on-creation


📝 Commits (2)

📊 Changes

20 files changed (+949 additions, -2280 deletions)

View changed files

📝 apps/remix/app/components/dialogs/document-move-to-folder-dialog.tsx (+31 -6)
📝 apps/remix/app/components/dialogs/folder-create-dialog.tsx (+55 -66)
📝 apps/remix/app/components/dialogs/folder-delete-dialog.tsx (+66 -56)
📝 apps/remix/app/components/dialogs/folder-move-dialog.tsx (+86 -62)
📝 apps/remix/app/components/dialogs/template-create-dialog.tsx (+7 -3)
apps/remix/app/components/dialogs/template-folder-create-dialog.tsx (+0 -165)
apps/remix/app/components/dialogs/template-folder-delete-dialog.tsx (+0 -163)
apps/remix/app/components/dialogs/template-folder-move-dialog.tsx (+0 -175)
apps/remix/app/components/dialogs/template-folder-settings-dialog.tsx (+0 -176)
📝 apps/remix/app/components/dialogs/template-move-to-folder-dialog.tsx (+31 -5)
📝 apps/remix/app/components/general/document/document-search.tsx (+6 -2)
📝 apps/remix/app/components/general/folder/folder-card.tsx (+134 -55)
apps/remix/app/components/general/folder/folder-grid.tsx (+249 -0)
📝 apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents._index.tsx (+30 -219)
📝 apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents.f.$folderId._index.tsx (+3 -372)
📝 apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents.folders._index.tsx (+85 -62)
📝 apps/remix/app/routes/_authenticated+/t.$teamUrl+/templates._index.tsx (+15 -221)
📝 apps/remix/app/routes/_authenticated+/t.$teamUrl+/templates.f.$folderId._index.tsx (+3 -367)
📝 apps/remix/app/routes/_authenticated+/t.$teamUrl+/templates.folders._index.tsx (+90 -67)
📝 packages/app-tests/e2e/folders/team-account-folders.spec.ts (+58 -38)

📄 Description

  • Changed "Root" to "Home" in multiple dialog components for uniformity.
  • Make the entire folder cards clickable.
  • Removed automated redirect to the folder when you create a new one
  • Added search on /folders pages and move dialogs

https://github.com/user-attachments/assets/a241c348-079f-4b78-8319-0730c8f6df21


🔄 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/1770 **Author:** [@catalinpit](https://github.com/catalinpit) **Created:** 5/2/2025 **Status:** ✅ Merged **Merged:** 6/19/2025 **Merged by:** [@dguyen](https://github.com/dguyen) **Base:** `main` ← **Head:** `fix/disable-folder-navigation-on-creation` --- ### 📝 Commits (2) - [`43d8ac8`](https://github.com/documenso/documenso/commit/43d8ac848e2eceabc1de2e2f1422ce90b9649247) fix: refactor folders - [`7cb5646`](https://github.com/documenso/documenso/commit/7cb564640b6c1677dc70495f81b3835ffb2904d9) fix: folders ### 📊 Changes **20 files changed** (+949 additions, -2280 deletions) <details> <summary>View changed files</summary> 📝 `apps/remix/app/components/dialogs/document-move-to-folder-dialog.tsx` (+31 -6) 📝 `apps/remix/app/components/dialogs/folder-create-dialog.tsx` (+55 -66) 📝 `apps/remix/app/components/dialogs/folder-delete-dialog.tsx` (+66 -56) 📝 `apps/remix/app/components/dialogs/folder-move-dialog.tsx` (+86 -62) 📝 `apps/remix/app/components/dialogs/template-create-dialog.tsx` (+7 -3) ➖ `apps/remix/app/components/dialogs/template-folder-create-dialog.tsx` (+0 -165) ➖ `apps/remix/app/components/dialogs/template-folder-delete-dialog.tsx` (+0 -163) ➖ `apps/remix/app/components/dialogs/template-folder-move-dialog.tsx` (+0 -175) ➖ `apps/remix/app/components/dialogs/template-folder-settings-dialog.tsx` (+0 -176) 📝 `apps/remix/app/components/dialogs/template-move-to-folder-dialog.tsx` (+31 -5) 📝 `apps/remix/app/components/general/document/document-search.tsx` (+6 -2) 📝 `apps/remix/app/components/general/folder/folder-card.tsx` (+134 -55) ➕ `apps/remix/app/components/general/folder/folder-grid.tsx` (+249 -0) 📝 `apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents._index.tsx` (+30 -219) 📝 `apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents.f.$folderId._index.tsx` (+3 -372) 📝 `apps/remix/app/routes/_authenticated+/t.$teamUrl+/documents.folders._index.tsx` (+85 -62) 📝 `apps/remix/app/routes/_authenticated+/t.$teamUrl+/templates._index.tsx` (+15 -221) 📝 `apps/remix/app/routes/_authenticated+/t.$teamUrl+/templates.f.$folderId._index.tsx` (+3 -367) 📝 `apps/remix/app/routes/_authenticated+/t.$teamUrl+/templates.folders._index.tsx` (+90 -67) 📝 `packages/app-tests/e2e/folders/team-account-folders.spec.ts` (+58 -38) </details> ### 📄 Description - Changed "Root" to "Home" in multiple dialog components for uniformity. - Make the entire folder cards clickable. - Removed automated redirect to the folder when you create a new one - Added search on `/folders` pages and `move` dialogs https://github.com/user-attachments/assets/a241c348-079f-4b78-8319-0730c8f6df21 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 20:31:17 +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#1836
No description provided.