Skip to content

Fix/chart screenshots not inserting#169

Open
mx00 wants to merge 5 commits into
Eleven-Trading:mainfrom
mx00:fix/chart-screenshots-not-inserting
Open

Fix/chart screenshots not inserting#169
mx00 wants to merge 5 commits into
Eleven-Trading:mainfrom
mx00:fix/chart-screenshots-not-inserting

Conversation

@mx00
Copy link
Copy Markdown

@mx00 mx00 commented May 25, 2026

Summary

Fixes chart screenshot upload and screenshot-to-trade linking issues in TradeNote.

Fixed

  • Fixed 413 PayloadTooLargeError during screenshot uploads
  • Increased Express body size limits to support larger chart screenshots
  • Fixed manual chart screenshots not attaching to the correct trade
  • Fixed fallback handling when tradeTagsDateUnix is undefined
  • Fixed screenshot naming to use the exact trade ID
  • Fixed manual uploads replacing existing screenshots instead of creating new ones

Technical Changes

  • Added 50mb Express JSON/urlencoded limits

  • Patched screenshot linking logic in:

    • src/utils/daily.js
    • src/utils/screenshots.js
    • src/views/Daily.vue
    • index.mjs
  • Added patch-tradenote-screenshot-upload.sh

Result

  • Automatic TradingView replay screenshots upload correctly
  • Manual chart screenshots attach correctly to trades
  • Multiple screenshots can now be attached to the same trade
  • Daily/Dashboard views continue working correctly

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 this pull request may close these issues.

1 participant