[PR #4838] [MERGED] Fix bug in ofx2json logic #5615

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

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/4838
Author: @jfdoming
Created: 4/18/2025
Status: Merged
Merged: 4/18/2025
Merged by: @jfdoming

Base: masterHead: jfdoming-patch-1


📝 Commits (4)

📊 Changes

4 files changed (+44 additions, -6 deletions)

View changed files

📝 packages/loot-core/src/server/transactions/import/__snapshots__/parse-file.test.ts.snap (+1 -1)
packages/loot-core/src/server/transactions/import/ofx2json.test.ts (+33 -0)
📝 packages/loot-core/src/server/transactions/import/ofx2json.ts (+4 -5)
upcoming-release-notes/4838.md (+6 -0)

📄 Description

Fixes a bug in the ofx2json logic where some HTML entities could get incorrectly translated.


🔄 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/4838 **Author:** [@jfdoming](https://github.com/jfdoming) **Created:** 4/18/2025 **Status:** ✅ Merged **Merged:** 4/18/2025 **Merged by:** [@jfdoming](https://github.com/jfdoming) **Base:** `master` ← **Head:** `jfdoming-patch-1` --- ### 📝 Commits (4) - [`b276407`](https://github.com/actualbudget/actual/commit/b276407b30e5828a2ed41eec757977b4c5eb98e0) Fix double unescape bug in ofx2json - [`91ddc6e`](https://github.com/actualbudget/actual/commit/91ddc6e280e9be9d785bb84eb75eb7aebd0d6612) Add test - [`02e68b0`](https://github.com/actualbudget/actual/commit/02e68b044bb6ce485c45aef159838480de089d42) Add release notes - [`a1fd3b5`](https://github.com/actualbudget/actual/commit/a1fd3b53652c5e625f6a9a04986f117497cf2726) Update snapshot ### 📊 Changes **4 files changed** (+44 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `packages/loot-core/src/server/transactions/import/__snapshots__/parse-file.test.ts.snap` (+1 -1) ➕ `packages/loot-core/src/server/transactions/import/ofx2json.test.ts` (+33 -0) 📝 `packages/loot-core/src/server/transactions/import/ofx2json.ts` (+4 -5) ➕ `upcoming-release-notes/4838.md` (+6 -0) </details> ### 📄 Description Fixes a bug in the ofx2json logic where some HTML entities could get incorrectly translated. --- <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:15:25 -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#5615