Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add BANK_INTEREST_ADJ and BANK_TRANSFER actions for Schwab CSV import #219

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kerrickstaley
Copy link

These actions appeared in my recent Schwab transactions CSV.

"Bank Interest Adj" seems to mean that the previous "Margin Interest" amount needed to be adjusted (in my case the adjustment amount was a penny).

"Bank Transfer" happens when you overdraw your checking account and the balance is automatically transferred from your brokerage account. (There doesn't seem to be a penalty for this so it seems like a nice way to simplify your accounts; just keep $0 in your checking account always and then when you pay bills it will automatically come out of your brokerage account).

Note: "Margin Interest" and "Bank Interest Adj" are treated as negative income, but I think it's more correct to treat them as an expense. This PR just preserves the existing behavior; I'll open a different PR to change it since it's orthogonal.

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.

1 participant