[PR #5387] [MERGED] Fix tags not syncing properly #5908

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

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/5387
Author: @youngcw
Created: 7/24/2025
Status: Merged
Merged: 7/25/2025
Merged by: @youngcw

Base: masterHead: youngcw/fix-tags-delete


📝 Commits (6)

📊 Changes

11 files changed (+111 additions, -121 deletions)

View changed files

📝 packages/desktop-client/src/components/tags/ManageTags.tsx (+23 -24)
📝 packages/desktop-client/src/components/tags/TagCreationRow.tsx (+4 -2)
📝 packages/desktop-client/src/components/tags/TagEditor.tsx (+4 -14)
📝 packages/desktop-client/src/components/tags/TagRow.tsx (+37 -69)
📝 packages/desktop-client/src/style/tags.ts (+2 -4)
packages/loot-core/migrations/1749799110001_tags_tombstone.sql (+5 -0)
📝 packages/loot-core/src/server/db/index.ts (+10 -3)
📝 packages/loot-core/src/server/db/types/index.ts (+2 -1)
📝 packages/loot-core/src/server/tags/app.ts (+17 -3)
📝 packages/loot-core/src/types/models/tags.ts (+1 -1)
upcoming-release-notes/5387.md (+6 -0)

📄 Description

So far this should fix deleting tags, and pogman-code removed the default tag setting that was breaking the schema checks.


🔄 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/5387 **Author:** [@youngcw](https://github.com/youngcw) **Created:** 7/24/2025 **Status:** ✅ Merged **Merged:** 7/25/2025 **Merged by:** [@youngcw](https://github.com/youngcw) **Base:** `master` ← **Head:** `youngcw/fix-tags-delete` --- ### 📝 Commits (6) - [`2421e32`](https://github.com/actualbudget/actual/commit/2421e32e085ce93a4bbffec6518b3c89f4260331) start - [`35b5248`](https://github.com/actualbudget/actual/commit/35b5248275622366834db2e42fc61ce4c7dafcca) migration - [`02e5683`](https://github.com/actualbudget/actual/commit/02e56839f167c974ff27d1a8379019af2c94626d) don't show deleted tags in list - [`bc0defc`](https://github.com/actualbudget/actual/commit/bc0defcaf36f5bc72783e89b4fc6ee6be458bb69) tag updates (#5389) - [`77d21a9`](https://github.com/actualbudget/actual/commit/77d21a917fdeebcbc09a16b45e5226aed1cdc52e) tags fixes (#5391) - [`6df577d`](https://github.com/actualbudget/actual/commit/6df577d1e19d56e095bfc603315e7407a8de1984) update note ### 📊 Changes **11 files changed** (+111 additions, -121 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/tags/ManageTags.tsx` (+23 -24) 📝 `packages/desktop-client/src/components/tags/TagCreationRow.tsx` (+4 -2) 📝 `packages/desktop-client/src/components/tags/TagEditor.tsx` (+4 -14) 📝 `packages/desktop-client/src/components/tags/TagRow.tsx` (+37 -69) 📝 `packages/desktop-client/src/style/tags.ts` (+2 -4) ➕ `packages/loot-core/migrations/1749799110001_tags_tombstone.sql` (+5 -0) 📝 `packages/loot-core/src/server/db/index.ts` (+10 -3) 📝 `packages/loot-core/src/server/db/types/index.ts` (+2 -1) 📝 `packages/loot-core/src/server/tags/app.ts` (+17 -3) 📝 `packages/loot-core/src/types/models/tags.ts` (+1 -1) ➕ `upcoming-release-notes/5387.md` (+6 -0) </details> ### 📄 Description So far this should fix deleting tags, and pogman-code removed the default tag setting that was breaking the schema checks. --- <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:20:44 -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#5908