[PR #2439] [CLOSED] fix: Don't allow duplicate cat-groups in budget at renaming #11357

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

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/2439
Author: @FlorianLang06
Created: 3/7/2024
Status: Closed

Base: masterHead: dup-category-group-rename


📝 Commits (6)

  • e540721 fix: Don't allow duplicate cat-groups in budget at renaming
  • 8153da1 fix: removed debug comment
  • e6531de Release notes
  • c4d1f48 fix: check in existingGroup query if the group is not the group to be update
  • e035b95 fix: mobile group add check for duplications
  • d54fc6d fix: linting

📊 Changes

4 files changed (+54 additions, -3 deletions)

View changed files

📝 packages/desktop-client/src/components/budget/MobileBudget.tsx (+16 -1)
📝 packages/desktop-client/src/components/budget/index.tsx (+20 -1)
📝 packages/loot-core/src/server/db/index.ts (+12 -1)
upcoming-release-notes/2439.md (+6 -0)

📄 Description

Checks:

  • UI Level check to check for checking duplicate group name (create and rename)
  • DB level check for the same (create and rename/update)

Inspired from PR #2262
Fixes: #2428


🔄 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/2439 **Author:** [@FlorianLang06](https://github.com/FlorianLang06) **Created:** 3/7/2024 **Status:** ❌ Closed **Base:** `master` ← **Head:** `dup-category-group-rename` --- ### 📝 Commits (6) - [`e540721`](https://github.com/actualbudget/actual/commit/e540721074e35d821ec514840311a6be3ed4cc43) fix: Don't allow duplicate cat-groups in budget at renaming - [`8153da1`](https://github.com/actualbudget/actual/commit/8153da139009b4f82e5b7740b45535b5972561ec) fix: removed debug comment - [`e6531de`](https://github.com/actualbudget/actual/commit/e6531de5f0d7c30113b8866f3546ce80c792b18e) Release notes - [`c4d1f48`](https://github.com/actualbudget/actual/commit/c4d1f48c93940a513afeb7b9a39488bf818f1153) fix: check in existingGroup query if the group is not the group to be update - [`e035b95`](https://github.com/actualbudget/actual/commit/e035b9535c1da0faa5175e9a6d36f3412d8023d0) fix: mobile group add check for duplications - [`d54fc6d`](https://github.com/actualbudget/actual/commit/d54fc6d1de09eb70714e60cbe1ac914683a86e42) fix: linting ### 📊 Changes **4 files changed** (+54 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/budget/MobileBudget.tsx` (+16 -1) 📝 `packages/desktop-client/src/components/budget/index.tsx` (+20 -1) 📝 `packages/loot-core/src/server/db/index.ts` (+12 -1) ➕ `upcoming-release-notes/2439.md` (+6 -0) </details> ### 📄 Description Checks: - UI Level check to check for checking duplicate group name (create and rename) - DB level check for the same (create and rename/update) Inspired from PR #2262 Fixes: #2428 --- <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-10 20:58: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#11357