-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
Swchab conversion error #75
Comments
Hi there! Looks like a header line is being processed as a transaction. Could you maybe provide a (partial) export so I can reproduce this and make a fix? Thanks! |
Here are the first few lines from one of the accounts:
Then there are more entries like the BNDX one, and the account total at the end that looks like this:
Exact dollar values and percentages are made up. |
Thank you! I will take a look at it this week and get back to you! |
@adrianbn The headers you posted don’t seem to match those of the sample export. The sample export is taken from the account history if I recall correctly (see the image in the readme). It seems like your export was taken from another screen and looks like it’s some kind of account position export (so the current status, not the transactions). Can you confirm this to me? |
Hi @dickwolff, Yes. I tried to import the positions because the history also was failing. This is what happens when I try to import the history:
Then when I try to import the output file to Ghostfolio
I think the line reporting isn't accurate (it may be skipping the headers and counting from 0?). I think this is the offending line
I tried with another of my Schwab accounts as well and the history export produces the following error:
I think the offending line is this
|
Gotcha, thanks for the detailed analysis. One more thing to check: which version of the tool are you running? |
Pulling from latest, `-v` reports 10.5.2.
…On Mon, May 20, 2024 at 10:55 AM Dick Wolff ***@***.***> wrote:
Gotcha, thanks for the detailed analysis. One more thing to check: which
version of the tool are you running?
—
Reply to this email directly, view it on GitHub
<#75 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA6NPRMIM5TOHGIPEDHMDR3ZDI2IRAVCNFSM6AAAAABH6VAM7OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRQHEZDQMRQGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
That would be npm, but I meant the version of Export to Ghostfolio. The container version from Docker hub |
|
Thanks! I'll probably have some time tonight to look into the issue and will get back to you when I have something for you to test 😅 |
I think the issue does relate to the "internal transfer" line, at least based on your information. Thanks again for the detailed infromation. I have added "transfer" to the ignore list, and that seems to work now. Can you pull the 0.13.2-beta image and let me know if this resolved the issue? After your confirmation I will merge the change. |
We're making progress but I am still having issues. One of my accounts has this problem:
Offending line is this
This likely needs to be treated as a cash deposit I am guessing. The other account shows no errors on the parsing, but it doesn't upload to ghostfolio. Pretty much every line errors out with:
|
The first error is a weird one. There are non-qualified dividends in the sample export as well, which I have got from another user when creating the Schwab converter. Those do have a symbol, like The last errors are due to the fact that the records don't have a type (buy, sell, etc) so that's probably a line that doesn't get converted correctly. What kind of record type do the lines have? So for |
Sorry for the late check-in, I have been busy lately. I added the |
Merged the changes, closing issue due to inactivity. Please open a new issue should there be something else. |
Tried with multiple accounts and I always get the following output:
Then when trying to import to Ghostfolio I get this error:
The text was updated successfully, but these errors were encountered: