[PR #5462] [MERGED] prevent closing an account into itself #5947

Closed
opened 2026-02-28 21:21:27 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/5462
Author: @matt-fidd
Created: 8/3/2025
Status: Merged
Merged: 8/5/2025
Merged by: @matt-fidd

Base: masterHead: close-account-select


📝 Commits (5)

  • cb60f0a prevent closing account into itself
  • d3e9378 note
  • cf56b80 prevent at API level too
  • 79e0fd1 Merge branch 'master' into close-account-select
  • dc8c99e Merge branch 'master' into close-account-select

📊 Changes

6 files changed (+21 additions, -1 deletions)

View changed files

📝 packages/desktop-client/src/components/autocomplete/AccountAutocomplete.tsx (+6 -1)
📝 packages/desktop-client/src/components/modals/AccountAutocompleteModal.tsx (+2 -0)
📝 packages/desktop-client/src/components/modals/CloseAccountModal.tsx (+2 -0)
📝 packages/desktop-client/src/modals/modalsSlice.ts (+1 -0)
📝 packages/loot-core/src/server/accounts/app.ts (+4 -0)
upcoming-release-notes/5462.md (+6 -0)

📄 Description

Fixes https://github.com/actualbudget/actual/issues/3516


🔄 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/5462 **Author:** [@matt-fidd](https://github.com/matt-fidd) **Created:** 8/3/2025 **Status:** ✅ Merged **Merged:** 8/5/2025 **Merged by:** [@matt-fidd](https://github.com/matt-fidd) **Base:** `master` ← **Head:** `close-account-select` --- ### 📝 Commits (5) - [`cb60f0a`](https://github.com/actualbudget/actual/commit/cb60f0ac3af38bf3b8dc624d9633ceaeba92c765) prevent closing account into itself - [`d3e9378`](https://github.com/actualbudget/actual/commit/d3e9378be79b2e7a9954c3ca5a20f29530a03e86) note - [`cf56b80`](https://github.com/actualbudget/actual/commit/cf56b8040da5ccf30d237bf6ac220b1dc5befcc2) prevent at API level too - [`79e0fd1`](https://github.com/actualbudget/actual/commit/79e0fd15dc29f2cd778b393598cc9dfa5b73a18c) Merge branch 'master' into close-account-select - [`dc8c99e`](https://github.com/actualbudget/actual/commit/dc8c99e1fe63ead54a31c91665e6a2e02fc93ae4) Merge branch 'master' into close-account-select ### 📊 Changes **6 files changed** (+21 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/autocomplete/AccountAutocomplete.tsx` (+6 -1) 📝 `packages/desktop-client/src/components/modals/AccountAutocompleteModal.tsx` (+2 -0) 📝 `packages/desktop-client/src/components/modals/CloseAccountModal.tsx` (+2 -0) 📝 `packages/desktop-client/src/modals/modalsSlice.ts` (+1 -0) 📝 `packages/loot-core/src/server/accounts/app.ts` (+4 -0) ➕ `upcoming-release-notes/5462.md` (+6 -0) </details> ### 📄 Description Fixes https://github.com/actualbudget/actual/issues/3516 --- <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-02-28 21:21:27 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#5947