Skip to content

fix(settings): FMP Test key uses EquitySearch, matching Intrinio - #1147

Open
bishallllllll wants to merge 1 commit into
TraderAlice:devfrom
bishallllllll:fix/market-data-fmp-test-endpoint
Open

fix(settings): FMP Test key uses EquitySearch, matching Intrinio#1147
bishallllllll wants to merge 1 commit into
TraderAlice:devfrom
bishallllllll:fix/market-data-fmp-test-endpoint

Conversation

@bishallllllll

Copy link
Copy Markdown

FMP Test-key endpoint: use EquitySearch (matches Intrinio)

Settings → Market Data → Advanced: the "Test FMP key" button probes a key via the Test-key endpoint. FMP was using EquityScreener {limit:1} while Intrinio already uses EquitySearch {query:'AAPL', limit:1}. This makes FMP consistent (and the screener endpoint is a heavier call for a key probe).

  • src/webui/routes/config.ts — one-line change in TEST_ENDPOINTS; EquitySearch is a registered FMP fetcher (packages/opentypebb/src/providers/fmp/index.ts:135)
  • Verified: root npx tsc --noEmit clean
  • Suggested labels: area:settings, area:market-data

Copilot AI lite review requested due to automatic review settings August 21, 2026 10:02
@vercel

vercel Bot commented Aug 21, 2026

Copy link
Copy Markdown

@bishallllllll is attempting to deploy a commit to the luokerenx4's Team Team on Vercel.

A member of the Team first needs to authorize it.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Settings → Market Data → Advanced “Test FMP key” probe to use the lighter EquitySearch model (aligned with Intrinio’s probe) instead of the heavier EquityScreener call.

Changes:

  • Switch FMP test-key endpoint model from EquityScreener to EquitySearch.
  • Provide a concrete query (AAPL) for the FMP EquitySearch probe to validate credentials.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/webui/routes/config.ts
@bishallllllll

Copy link
Copy Markdown
Author

@copilot Fix the code for all comments in this review thread.

When a review comment includes a suggested change, apply the suggestion exactly.

Do not make changes beyond what is described in the linked review thread.

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.

2 participants