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

Crypto CSV from CDC fails to import #3056

Closed
Blackswan122 opened this issue Jun 6, 2021 · 10 comments
Closed

Crypto CSV from CDC fails to import #3056

Blackswan122 opened this issue Jun 6, 2021 · 10 comments
Labels
file import Importing data from a specific service given a file (e.g. CSV)

Comments

@Blackswan122
Copy link

Hi,
I tried to import import the CDC file ...Crypto.. it failed.

The other two Fiat and Cards worked fine.

CDC Rotki 3

CDC Rotki 2
CDC Rotki

I tried to send Excel CSV but it says this file not supported.

App Version: 1.17.2
Frontend Version: 1.17.2
Platform: win32 x64 10.0.19042
Electron: 12.0.7

Operating system:
Rotki version:

@yabirgb
Copy link
Member

yabirgb commented Jun 6, 2021

In order:

  • First warning might be because you have an empty line in the CSV files
  • About viban_* I don't see any code in our side to handle this kind of transactions so we need to add support for it. It would be necessary that you provided us with faked data for this type of rows.

About that error I tried with the CSV in the screenshot and is not the cause. Could you also give me fake data to check what could it be?

@yabirgb yabirgb added the file import Importing data from a specific service given a file (e.g. CSV) label Jun 6, 2021
@Blackswan122
Copy link
Author

Blackswan122 commented Jun 6, 2021 via email

@yabirgb
Copy link
Member

yabirgb commented Jun 6, 2021

Sorry, with fake data I meant a CSV file that didn't contain information about you (values) but is similar to what you see.
In this case headers are not the problem but some row, that is why you see missing entry for 'debited'.

With the viban_ I was referring to the warnings that you see in the UI. They mention viban_deposit, viban_purchase, viban_card_top_up. Those are actions that we don't have any information about as we haven't seen them before.

@Blackswan122
Copy link
Author

Blackswan122 commented Jun 6, 2021 via email

@yabirgb
Copy link
Member

yabirgb commented Jun 6, 2021

Sorry @Blackswan122 I believe I didn't explain well. I meant to say that if you could send either to me in discord or here the csv file that is failing so I can check why this error is showing with random numbers so no information about you is revealed.

@LefterisJP
Copy link
Member

@Blackswan122 CSV formats are experimental because crypto.com, nexo and all other CSV services changes them every other day. If you want us to fix this you will need to send us an example CSV file. You are also in discord, you can DM yabir.

@yabirgb This is why I keep saying that CSV support is hard. There is no spec. They just change it all the time. We will need to implement a generic CSV import so users can fix these stuff on their own: #2270

@LefterisJP
Copy link
Member

Any updates here? With absence of more information I am going to close the issue.

@yabirgb
Copy link
Member

yabirgb commented Jul 25, 2021

Shame on me. Not for the moment. Now that I have more time I'll take a look at it ASAP. We fixed an issue related to nexo already but there are still one file failing

@LefterisJP
Copy link
Member

Yes but do you have the CSV that lead to the failure? Without it you can't debug this.

@LefterisJP
Copy link
Member

Fixed by #3424

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
file import Importing data from a specific service given a file (e.g. CSV)
Projects
None yet
Development

No branches or pull requests

3 participants