[PR #1943] [MERGED] feat: reset user 2fa from admin panel #1946

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

📋 Pull Request Information

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

Base: mainHead: feat/reset-2fa-via-admin-panel


📝 Commits (8)

  • e777c86 feat: reset user 2fa from admin
  • f71ac8d chore: backend logic
  • 4bd8afc Merge branch 'main' into feat/reset-2fa-via-admin-panel
  • 4155db6 chore: cleanup
  • 94b7461 Merge branch 'feat/reset-2fa-via-admin-panel' of github.com:documenso/documenso into feat/reset-2fa-via-admin-panel
  • 3093656 chore: last changes
  • 3cb2905 chore: rename id to userId
  • e2dea61 Merge branch 'main' into feat/reset-2fa-via-admin-panel

📊 Changes

5 files changed (+227 additions, -2 deletions)

View changed files

apps/remix/app/components/dialogs/admin-user-reset-two-factor-dialog.tsx (+159 -0)
📝 apps/remix/app/routes/_authenticated+/admin+/users.$id.tsx (+4 -2)
packages/trpc/server/admin-router/reset-two-factor-authentication.ts (+50 -0)
packages/trpc/server/admin-router/reset-two-factor-authentication.types.ts (+10 -0)
📝 packages/trpc/server/admin-router/router.ts (+4 -0)

📄 Description

Description

Allow admins to disable the two factor authentication from the admin panel.

Testing Performed

Tested it locally.

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.

🔄 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/1943 **Author:** [@catalinpit](https://github.com/catalinpit) **Created:** 8/6/2025 **Status:** ✅ Merged **Merged:** 8/19/2025 **Merged by:** [@dguyen](https://github.com/dguyen) **Base:** `main` ← **Head:** `feat/reset-2fa-via-admin-panel` --- ### 📝 Commits (8) - [`e777c86`](https://github.com/documenso/documenso/commit/e777c860ebff9afb5a507e49a15ddaed01e2a26e) feat: reset user 2fa from admin - [`f71ac8d`](https://github.com/documenso/documenso/commit/f71ac8da4a929e5d80567d3a207bdf9348e89e35) chore: backend logic - [`4bd8afc`](https://github.com/documenso/documenso/commit/4bd8afc50984e271e68b8b12abe1ae9926bf5ec7) Merge branch 'main' into feat/reset-2fa-via-admin-panel - [`4155db6`](https://github.com/documenso/documenso/commit/4155db6ddb55631790ebca62c1ae428d0ecc31df) chore: cleanup - [`94b7461`](https://github.com/documenso/documenso/commit/94b7461979cd3966fc23ab1d2066db3fc9001982) Merge branch 'feat/reset-2fa-via-admin-panel' of github.com:documenso/documenso into feat/reset-2fa-via-admin-panel - [`3093656`](https://github.com/documenso/documenso/commit/3093656c67fe325a8a40af39245b41a9f75bd4cf) chore: last changes - [`3cb2905`](https://github.com/documenso/documenso/commit/3cb2905654316d3dd5caea6aa0790676500c7295) chore: rename id to userId - [`e2dea61`](https://github.com/documenso/documenso/commit/e2dea61ddc33b2bbf01636e2e9c654160b6efbcb) Merge branch 'main' into feat/reset-2fa-via-admin-panel ### 📊 Changes **5 files changed** (+227 additions, -2 deletions) <details> <summary>View changed files</summary> ➕ `apps/remix/app/components/dialogs/admin-user-reset-two-factor-dialog.tsx` (+159 -0) 📝 `apps/remix/app/routes/_authenticated+/admin+/users.$id.tsx` (+4 -2) ➕ `packages/trpc/server/admin-router/reset-two-factor-authentication.ts` (+50 -0) ➕ `packages/trpc/server/admin-router/reset-two-factor-authentication.types.ts` (+10 -0) 📝 `packages/trpc/server/admin-router/router.ts` (+4 -0) </details> ### 📄 Description ## Description Allow admins to disable the two factor authentication from the admin panel. ## Testing Performed Tested it locally. ## Checklist <!--- Please check the boxes that apply to this pull request. --> <!--- You can add or remove items as needed. --> - [x] 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. - [x] I have followed the project's coding style guidelines. - [ ] I have addressed the code review feedback from the previous submission, if applicable. --- <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:46 +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#1946
No description provided.