[PR #1972] [MERGED] [Maintenance] Update ConfirmCategoryDelete, GoCardlessExternalMsg, ManageRulesModal to tsx #18414

Closed
opened 2026-04-14 20:42:52 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/1972
Author: @MikesGlitch
Created: 11/25/2023
Status: Merged
Merged: 11/26/2023
Merged by: @MatissJanis

Base: masterHead: maintenance/modals-to-tsx


📝 Commits (5)

📊 Changes

6 files changed (+76 additions, -22 deletions)

View changed files

📝 packages/desktop-client/src/components/modals/ConfirmCategoryDelete.tsx (+16 -5)
📝 packages/desktop-client/src/components/modals/GoCardlessExternalMsg.tsx (+28 -14)
📝 packages/desktop-client/src/components/modals/ManageRulesModal.tsx (+11 -2)
📝 packages/loot-core/src/types/models/gocardless.d.ts (+10 -0)
📝 packages/loot-core/src/types/server-handlers.d.ts (+5 -1)
upcoming-release-notes/1972.md (+6 -0)

📄 Description

Update ConfirmCategoryDelete, GoCardlessExternalMsg, ManageRulesModal to tsx

https://github.com/actualbudget/actual/issues/1483


🔄 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/actualbudget/actual/pull/1972 **Author:** [@MikesGlitch](https://github.com/MikesGlitch) **Created:** 11/25/2023 **Status:** ✅ Merged **Merged:** 11/26/2023 **Merged by:** [@MatissJanis](https://github.com/MatissJanis) **Base:** `master` ← **Head:** `maintenance/modals-to-tsx` --- ### 📝 Commits (5) - [`651b513`](https://github.com/actualbudget/actual/commit/651b513ddf6c74af3702df60da11cece5bdcdbe4) update some modals to tsx - [`73048a0`](https://github.com/actualbudget/actual/commit/73048a05c1ab33ba8d0f7f63efa811f74fbdd6ef) updating managerulesmodal to tsx - [`4dff896`](https://github.com/actualbudget/actual/commit/4dff8966284fd42a45aa5f2cbbd5c635c73f8c73) remove debugger statement - [`aaae4fd`](https://github.com/actualbudget/actual/commit/aaae4fdef06b5f91e15544b2c39f1fb1d3f1e1b9) remove non-prop - [`b38b8b4`](https://github.com/actualbudget/actual/commit/b38b8b4d45a397c759d26111a37b40987d574e40) Merge branch 'master' of https://github.com/MikesGlitch/actual into maintenance/modals-to-tsx ### 📊 Changes **6 files changed** (+76 additions, -22 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/modals/ConfirmCategoryDelete.tsx` (+16 -5) 📝 `packages/desktop-client/src/components/modals/GoCardlessExternalMsg.tsx` (+28 -14) 📝 `packages/desktop-client/src/components/modals/ManageRulesModal.tsx` (+11 -2) 📝 `packages/loot-core/src/types/models/gocardless.d.ts` (+10 -0) 📝 `packages/loot-core/src/types/server-handlers.d.ts` (+5 -1) ➕ `upcoming-release-notes/1972.md` (+6 -0) </details> ### 📄 Description <!-- Thank you for submitting a pull request! Make sure to follow the instructions to write release notes for your PR — it should only take a minute or two: https://github.com/actualbudget/docs#writing-good-release-notes --> Update ConfirmCategoryDelete, GoCardlessExternalMsg, ManageRulesModal to tsx https://github.com/actualbudget/actual/issues/1483 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-14 20:42:52 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#18414