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

Problem with OTC trades #469

Closed
smurfer opened this issue Aug 17, 2019 · 3 comments · Fixed by #466
Closed

Problem with OTC trades #469

smurfer opened this issue Aug 17, 2019 · 3 comments · Fixed by #466

Comments

@smurfer
Copy link

smurfer commented Aug 17, 2019

Problem Definition

entered an OTC trade: EUR_BTC, type "buy" (did not change the default drop down), amount 0.5, rate 351, time some date in 2013.

the entry shows up in the OTC trades list, but with an empty "type", and when I open the OTC trades list from the menu again, a windows error dialog pops up (apparently from the db system, not rotki; screen shot see link:
image
It seems that the type is not stored in the db, but is tried to be retrieved, which fails.

two further issues:

  • I did not find a way to delete transactions.
  • The "Exchange Trades" list is empty

System Description

Windows 1903

@smurfer smurfer changed the title Probelm with OTC trades Problem with OTC trades Aug 17, 2019
@LefterisJP
Copy link
Member

Hey, thanks for opening the issue!

I will look into the problem you reported with the OTC trades.

(apparently from the db system, not rotki; screen shot see link:

Datatable is actually in the node.js frontend. It's what creates those tables. Let me have a look and try to reproduce your problem.

As for the other comments:

I did not find a way to delete transactions.

By right clicking on any transaction on the table below you should get a context menu with 2 options. "Edit" or "Delete".

I also just noticed this is not in the docs: https://rotkehlchen.readthedocs.io/en/latest/usage_guide.html#manually-adding-trades-or-taxable-events

So I will add this information there.

The "Exchange Trades" list is empty

That is actually a long standing TODO and has an issue: #284

It's very easy to do but I am waiting for the front-end rewrite from @kelsos to proceed with adding this as I don't want to mess with the frontend UI before he is finished.

@kelsos
Copy link
Member

kelsos commented Aug 21, 2019

@LefterisJP I actually encountered the OTC trades issue when running the tests for the date conversion. I will actually fix it in the same PR.

@LefterisJP
Copy link
Member

Oh wait so you are fixing this also in that PR? that's nice. Is there a test for it?

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 a pull request may close this issue.

3 participants