Skip to content

BUG: Tagged trades getting deleted if day contains tag #129

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

Open
debegr92 opened this issue Mar 17, 2025 · 6 comments
Open

BUG: Tagged trades getting deleted if day contains tag #129

debegr92 opened this issue Mar 17, 2025 · 6 comments

Comments

@debegr92
Copy link

Problem
Trades get sometimes deleted from the overview and all statistics.
This problem happend a lot to me before I figured out, what causes this.

How to reproduce

  1. Add trades
  2. Add a tag for the trading day (screen 1)
  3. Open a trade
  4. Add a tag for the trade (screen 2)
  5. Reload the page
  6. The tagged trade is now no longer there

Image

Image

Image

Some debug output

Image

@C8opmBM
Copy link

C8opmBM commented Mar 19, 2025

I cannot reproduce this.
For me, tagged trades are visible all the time.

@debegr92
Copy link
Author

I cannot reproduce this.

Are you using TradeNote from the official docker image or building the container from the source?

I even tested tagging first the day then the trade and vice versa. Both leads to missing trades.

@C8opmBM
Copy link

C8opmBM commented Mar 20, 2025

Using the official docker, hosted on my server.
Been tinkering a lot with importing trades, etc, never missing tagged trades.
Will send you tomorrow my csv (using Template), and you can test with it. Now Im on the road.

@debegr92
Copy link
Author

Using the official docker, hosted on my server.

I am using the official docker on my server and building the container from source for local development. Both times I can reproduce this behavior.

@C8opmBM
Copy link

C8opmBM commented Mar 21, 2025

Unfortunately I cannot provide you with sample for you to test now, as reimport does not seem to work, see #131
I deleted the trading logs, will get back with a sample in the coming days.

@C8opmBM
Copy link

C8opmBM commented Apr 8, 2025

You are right, I now can reproduce this.

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

No branches or pull requests

2 participants