feat: disable paper trading UI - #530
Merged
Merged
Conversation
Paper trading beside backtests is too much surface for now. One flag, PAPER_TRADING_ENABLED = false in app.js (the paper inputs in app.html ship disabled to match), turns off every paper surface in the dashboard: - Paper Trading subtab greyed out; ?view=paper and saved nav state land on My Agents. - Agent cards show only the Backtesting capital; no Run Paper Trading button, no PAPER TRADING status. - My Portfolio overview + Capital Allocation pie hidden on My Agents. - Configure: paper field disabled; the PATCH omits cash_allocation, so existing sleeves are left untouched. - Create flows hide the paper field and post cash_allocation: 0, so new agents stop silently reserving $1,000 of an invisible balance. - Copy no longer mentions paper trading in the backtest modal/auth. Backend /paper/* routes are unchanged. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Turns off every paper-trading surface in the dashboard behind one flag,
PAPER_TRADING_ENABLED = false(app.js). The paper inputs inapp.htmlshipdisabledto match, so re-enabling means flipping both.?view=paper/ saved nav state now land on My Agents. Cards show only the Backtesting capital, with no Run Paper Trading button and no PAPER TRADING status. The My Portfolio overview and Capital Allocation pie are hidden.cash_allocationso existing sleeves are left untouched.cash_allocation: 0. Otherwise each would quietly reserve $1,000 of a balance nobody can see, until creates start failing with "Insufficient unallocated cash". Backtests still default to $1,000.UI-only:
/paper/*routes and server-side starter provisioning ($1,000 sleeves) are unchanged.Not covered: Home still shows the My Portfolio panel and the "Paper trades today" stat.
Tests: full backend suite 5456 passed / 122 skipped;
node --checkclean onapp.jsandagent-editor.js. Not checked in a browser.🤖 Generated with Claude Code