-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
Transaction XTB error #116
Comments
Thanks for reporting this. I will look into it when I have time! |
@rubengr I've just looked at the issue. The I assume this record is not standalone, but rather connected to another BUY/SELL record. Could you maybe send me both lines from your export, so I can use that to create a working solution? |
@rubengr Could you check my latest message? Thanks! |
sorry for late reply. I think is related to taxes of dividends. For example:
|
Sorry for the late reply. I was busy finishing things up last year. Now I have more time for this project and I will look into this issue today! |
@rubengr After looking into it, it seems the XTB converter did not have support for dividends at all. I've added this just now, and used your input as test data. Can you test if this works for you? You can clone this branch and run it yourself, or use this Docker image to verify. Please let me know if all works as expected so I can merge this. |
Get this error:
|
That's a shame. Can you maybe provide me with the input you are trying with? That way I can better reproduce the issue myself. Thanks! |
Thanks! I will look into it and get back to you! |
I just took a look at the file you provided, but are you sure this ia a XTB export? It looks more like a DEGIRO file. The file you provided has a lot more headers then the one in my XTB sample (which is just |
@toffee I saw you were also looking into adding dividends to the XTB converter. Could you maybe try my fix, to verify if it all processes correctly? You can use this Docker image, or clone the branch if you want to run locally. Thanks a lot! |
I prepared a test export. Running the code from the branch on this input file give me an error (on record 9):
Additionally, seems the yahoo finance did not find correctly some stock symbols (for FLOA.UK wrongly find PFLT or for CSPX.UK find CSPX.L or for SPY5.UK find SPY508E-DC179.MX). Probably this should be discussed separately if it's an issue (or is it wrong configuration on my side!?) |
Thanks for the input! The issue seems to be with When I search on Yahoo Finance for |
@toffee I just pushed some changes, I also added some records of your test export. Can you verify if this works for you and all records are processed correctly? |
With the latest changes the generation of json file ends but the content is not ok. See bellow the generated json (3 activities) for these 6 cvs records:
As you can see, for the first two activities the fee is wrong and for the third activity has quantity and unitPrice null. To change the UK to L don't think it's correct - the symbol as it's come from XTB is correct (except the ones with US). I guess the removal of ".country" should be done only for US (in case of yahoo finance). |
I will look into the tax issue. But when I remove the .UK (as previoulsy), then you don't get completely different symbols right? The symbols that XTB uses are not known in Yahoo Finance. So with this replacement of |
Thank you. My previous assumption was wrong. I'm getting the correct security with your change from UK to L. |
@dickwolff you are completely right. Sorry for my mistake.
|
No problem @rubengr. I will investigate further and let you know what i find! |
Hi
I'm testing you code, but get this error:
The text was updated successfully, but these errors were encountered: