[PR #4224] [MERGED] fix deleting rules considering the applied filters #23881

Closed
opened 2026-04-16 17:43:00 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/4224
Author: @harrydigos
Created: 1/23/2025
Status: Merged
Merged: 2/5/2025
Merged by: @matt-fidd

Base: masterHead: digos/delete-selected-rules


📝 Commits (5)

  • a7eff8d fix: combine selected and filtered rules when deleting
  • 240e48d chore: add release note
  • 522158d fix: update deps array
  • 56012cc refactor: replace intersection
  • a8801f2 fix: keep selected after filters

📊 Changes

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

View changed files

📝 packages/desktop-client/src/components/ManageRules.tsx (+9 -5)
upcoming-release-notes/4224.md (+6 -0)

📄 Description

Fixes #3366


🔄 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/4224 **Author:** [@harrydigos](https://github.com/harrydigos) **Created:** 1/23/2025 **Status:** ✅ Merged **Merged:** 2/5/2025 **Merged by:** [@matt-fidd](https://github.com/matt-fidd) **Base:** `master` ← **Head:** `digos/delete-selected-rules` --- ### 📝 Commits (5) - [`a7eff8d`](https://github.com/actualbudget/actual/commit/a7eff8df604484b1548194834b498c218ead6ac5) fix: combine selected and filtered rules when deleting - [`240e48d`](https://github.com/actualbudget/actual/commit/240e48d7e69b1f8ba74228903d31908a56cc1d74) chore: add release note - [`522158d`](https://github.com/actualbudget/actual/commit/522158d13713d7271fe59272a9f9eb7bee475bd0) fix: update deps array - [`56012cc`](https://github.com/actualbudget/actual/commit/56012cce59f6113b19f139bcf56dea6633617a7d) refactor: replace intersection - [`a8801f2`](https://github.com/actualbudget/actual/commit/a8801f20edbc03af358ed719989a2deab3c5110d) fix: keep selected after filters ### 📊 Changes **2 files changed** (+15 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/ManageRules.tsx` (+9 -5) ➕ `upcoming-release-notes/4224.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 --> Fixes #3366 --- <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-16 17:43:00 -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#23881