[PR #5464] [MERGED] make bulk notes prepend/append behaviour consistent with rules #5948

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

📋 Pull Request Information

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

Base: masterHead: batch-note-edit


📝 Commits (4)

  • 8bfa579 make bulk notes prepend/append behaviour consistent with rules
  • 200a7be note
  • b046616 typecheck
  • 688a54b Merge branch 'master' into batch-note-edit

📊 Changes

2 files changed (+8 additions, -2 deletions)

View changed files

📝 packages/desktop-client/src/hooks/useTransactionBatchActions.ts (+2 -2)
upcoming-release-notes/5464.md (+6 -0)

📄 Description

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

I went this direction because it's less disruptive than changing the rules behaviour and breaking existing rules.

There's probably a discussion here as to which behaviour makes most sense, but this gives the user complete control and lets them choose whether to add the space or not so I'm in favour.


🔄 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/5464 **Author:** [@matt-fidd](https://github.com/matt-fidd) **Created:** 8/3/2025 **Status:** ✅ Merged **Merged:** 8/6/2025 **Merged by:** [@matt-fidd](https://github.com/matt-fidd) **Base:** `master` ← **Head:** `batch-note-edit` --- ### 📝 Commits (4) - [`8bfa579`](https://github.com/actualbudget/actual/commit/8bfa579800a5c289aba4b8721b87e9e3ec3696ea) make bulk notes prepend/append behaviour consistent with rules - [`200a7be`](https://github.com/actualbudget/actual/commit/200a7bea25f7e12779a18a29ec66b80c39e49400) note - [`b046616`](https://github.com/actualbudget/actual/commit/b046616378a9d572f92ace20a2b53ca01c1a8a41) typecheck - [`688a54b`](https://github.com/actualbudget/actual/commit/688a54ba995b801c5102a350882b2900707128d2) Merge branch 'master' into batch-note-edit ### 📊 Changes **2 files changed** (+8 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/hooks/useTransactionBatchActions.ts` (+2 -2) ➕ `upcoming-release-notes/5464.md` (+6 -0) </details> ### 📄 Description Fixes https://github.com/actualbudget/actual/issues/5271 I went this direction because it's less disruptive than changing the rules behaviour and breaking existing rules. There's probably a discussion here as to which behaviour makes most sense, but this gives the user complete control and lets them choose whether to add the space or not so I'm in favour. --- <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:28 -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#5948