Converts html special characters in ofx values to plaintext #2364
Conversation
- Apply during ofx/qfx import
✅ Deploy Preview for actualbudget ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Bundle Stats — desktop-clientHey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle. As this PR is updated, I'll keep you updated on how the bundle size is impacted. Total
Changeset No files were changed View detailed bundle breakdownAdded No assets were added Removed No assets were removed Bigger No assets were bigger Smaller No assets were smaller Unchanged
|
Bundle Stats — loot-coreHey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle. As this PR is updated, I'll keep you updated on how the bundle size is impacted. Total
Changeset
View detailed bundle breakdownAdded No assets were added Removed No assets were removed Bigger
Smaller No assets were smaller Unchanged No assets were unchanged |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
twk3
changed the title
trafico-bot
bot
added
✨ Merged
…dget#2364) * Converts html special characters in ofx values to plaintext - Apply during ofx/qfx import
Summary
Converts things like
&and<back into plaintext when encountered in qfx/ofx memo or name fields during import.Issue
Fixes #1872
Details
One downside is that users that have payees that are impacted by these characters are going to all of a sudden have additional non-matched payees added when importing with this in place.
Though this could just as easily happen if the banks with these issues fixed their exporters at any point to only have plaintext to begin with.
This change also happens to apply the ampersand replacement in an order that fixes order replacement issues in the source data. (where they did their ampersand replacement after already doing their LT/GT replacement.)
Tests
The example file provided in the issue has been added here as a new test mock.