Skip to content

chore: use "trade" consistently in prediction-market copy - #576

Merged
DavidMinarsch merged 1 commit into
mainfrom
chore/terminology-trades
Sep 5, 2026
Merged

chore: use "trade" consistently in prediction-market copy#576
DavidMinarsch merged 1 commit into
mainfrom
chore/terminology-trades

Conversation

@DavidMinarsch

Copy link
Copy Markdown
Contributor

Summary

Aligns the user-facing prediction-market copy on "trade", matching the terminology the site already uses elsewhere — the Omenstrat "Makes Smart Trades" step, the "places trades" description beneath it, and MIN_TRADES_FOR_ROI_DISPLAY.

Copy only; no logic, no metric definitions, no rendered numbers change.

Where

8 text sites across 6 routes:

Route File
/agents/omenstrat Agents/PredictionAgentsPage/FAQ.tsx
/agents/ai-mechs PredictionAgentsTable/index.tsx
/mech-marketplace MechMarketplacePage/BenefitFromMM.tsx
/agent-economies/predict AgentEconomies/PredictPage/PlatformActivitySection.tsx, NetPositiveRateCard.tsx
/agent-economies/explorer ExplorerPage/index.tsx
/data DataPage/OmenstratAccuracy.tsx, PolystratAccuracy.tsx

Prettier reflowed a few of the edited paragraphs; those line moves are formatting only.

Deliberately unchanged

  • The bets entity in the GraphQL snippets printed on /data. Those blocks are the literal copy-pasteable queries against the Omen subgraph and the Polymarket squid — bets is the upstream schema's entity name, so renaming it would break the query.
  • Internal identifiers that mirror that schema — types, variables and comments in common-util/api/predict/*, common-util/api/explorer.ts and common-util/graphql/queries.ts.
  • data/timeline.json — the one occurrence there is inside a quoted external article title.

Also

Removes data/opportunities.json, which is imported nowhere and renders on no page. Note this leaves the 16 files under public/images/opportunities/ orphaned — they were referenced only from that JSON. Left in place here since public assets can be linked externally; happy to remove them in a follow-up.

Verification

Rendered each of the 6 routes from the dev server and grepped the served HTML for \b(bet|bets|betting|bettor)\b:

  • /agents/ai-mechs, /mech-marketplace, /agent-economies/predict, /agent-economies/explorer — 0 matches.
  • /agents/omenstrat — only the smart-bets.png image URL in a srcset (asset filename; the visible heading and alt are both "Makes Smart Trades").
  • /data — only the raw GraphQL text noted above.

yarn lint clean.

🤖 Generated with Claude Code

https://claude.ai/code/session_01D3mccotoPBzhS53E9LRS3z

Aligns the user-facing prediction-market wording on "trade", matching the
terminology already used elsewhere on the site — the Omenstrat "Makes
Smart Trades" step, the "places trades" description beneath it, and
MIN_TRADES_FOR_ROI_DISPLAY.

Prose only, across 8 sites on 6 routes: /agents/omenstrat,
/agents/ai-mechs, /mech-marketplace, /agent-economies/predict,
/agent-economies/explorer and /data. The `bets` entity in the GraphQL
snippets printed on /data belongs to the upstream Omen and Polymarket
schemas and is unchanged, as are the internal identifiers that mirror it.

Also drops data/opportunities.json, which is imported nowhere.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01D3mccotoPBzhS53E9LRS3z
@vercel

vercel Bot commented Sep 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
olas-website Ready Ready Preview Sep 5, 2026 6:52pm UTC

Request Review

@DavidMinarsch
DavidMinarsch merged commit f855d95 into main Sep 5, 2026
10 checks passed
@DavidMinarsch
DavidMinarsch deleted the chore/terminology-trades branch September 5, 2026 18:55
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