Skip to content

Conversation

devin-ai-integration[bot]
Copy link
Contributor

Description

Adds support for searching multiple price feeds simultaneously in the Lazer price feed IDs table. Users can now enter comma or space-separated search terms to find multiple feeds at once.

Example usage:

  • BTC ETH - shows feeds matching either BTC or ETH
  • BTC, ETH, SOL - shows feeds matching any of the three terms
  • Single terms continue to work as before

Type of Change

  • Page update/improvement
  • New Page
  • Fix typo/grammar
  • Restructure/reorganize content
  • Update links/references
  • Other (please describe):

Areas Affected

  • /pages/lazer/price-feed-ids.mdx - Lazer price feed IDs page
  • components/LazerPriceIdTable.tsx - Search filtering logic

Implementation Details

The search now:

  1. Splits input by commas or spaces using regex /[,\s]+/
  2. Trims and lowercases each term
  3. Returns feeds matching any of the search terms (OR logic)
  4. Maintains backward compatibility with single-term searches

Checklist

  • I ran pre-commit run --all-files to check for linting errors (npm run lint passed)
  • I have reviewed my changes for clarity and accuracy
  • All links are valid and working
  • Images (if any) are properly formatted and include alt text (N/A)
  • Code examples (if any) are complete and functional
  • Content follows the established style guide
  • Changes are properly formatted in Markdown
  • Preview renders correctly in development environment (CORS issue prevented full testing with live data)

Related Issues

Per request in Slack thread: https://dourolabs.slack.com/archives/C06Q9S66BE2/p1760448200091969

Additional Notes

Testing limitation: Local development testing encountered CORS issues with the Lazer API endpoint, preventing full end-to-end verification. The code logic is sound and linting passes, but production testing is recommended.

Search behavior: Uses OR logic - a feed is shown if it matches ANY of the search terms. This seemed like the most intuitive behavior for multi-term search.

Link to Devin run: https://app.devin.ai/sessions/f03f3f87ab7649ca974ea65a2c699336

Contributor Information

- Support comma or space-separated search terms
- Match any feed that includes at least one of the search terms
- Update placeholder text to indicate multi-search capability

Co-Authored-By: Jayant <jayantkrishnamurthy@gmail.com>
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Copy link

vercel bot commented Oct 14, 2025

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

Project Deployment Preview Comments Updated (UTC)
docs Ready Ready Preview Comment Oct 14, 2025 6:53pm
documentation Error Error Oct 14, 2025 6:53pm

Co-Authored-By: Jayant <jayantkrishnamurthy@gmail.com>
Co-Authored-By: Jayant <jayantkrishnamurthy@gmail.com>
@jayantk jayantk merged commit a5e6299 into main Oct 14, 2025
3 of 5 checks passed
@jayantk jayantk deleted the devin/1760467391-multi-feed-search branch October 14, 2025 19:17
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