Skip to content

Add support for dividend reinvestments and stock distributions#1

Open
oyarzun wants to merge 1 commit into
zhou13:masterfrom
oyarzun:zhou13/add-types
Open

Add support for dividend reinvestments and stock distributions#1
oyarzun wants to merge 1 commit into
zhou13:masterfrom
oyarzun:zhou13/add-types

Conversation

@oyarzun

@oyarzun oyarzun commented Dec 24, 2025

Copy link
Copy Markdown

Thanks for your work on improving the Fidelity conversion plugin.

Hopefully Fidelity will offer an OFX download, but in the mean time here are some additional changes to support converting the CSV files.

Note: the stock distributions require an update to ofxstatement. I will push another PR to your repo.

@zhou13

zhou13 commented Mar 30, 2026

Copy link
Copy Markdown
Owner

Sorry for missing your PR. Could you explain the TRANSFER entry? I asked ChatGPT and it says

In an OFX investment statement, TRANSFER usually is not the value of <TRNTYPE>

If it is for stock distributions, I suppose

set_income("DIST")

will be more reasonable? Correct me if I am wrong, but according to my understanding, distribution is very similar to diviends.

@zhou13

zhou13 commented Mar 30, 2026

Copy link
Copy Markdown
Owner

Updated comments above.

@oyarzun

oyarzun commented Mar 31, 2026

Copy link
Copy Markdown
Author

A TRANSFER in the OFX specification can be used for a stock distribution since the stock is transferred into the brokerage account.

see kedder/ofxstatement#378

   set_income("DIST")

That would set the INCOMETYPE to DIST which is not part of the OFX specification.
https://schemas.liquid-technologies.com/ofx/2.1.1/?page=incomeenum.html

@zhou13

zhou13 commented Mar 31, 2026

Copy link
Copy Markdown
Owner

Thank you for the clarification. In https://dev.wealthaccess.com/wp-content/uploads/2024/02/QW_DataMappingGuide.pdf (Page 52), it suggests using RETOFCAP + BUYSTOCK for a spin-off, which I think is what you encountered. Let me know what you think. I can also merge this as is, as RETOFCAP might not be well supported by downstream applications.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants