Skip to content

Fix profile comment search — wire up term param to API and backend query#3695

Merged
IanPhilips merged 1 commit intomainfrom
fix/profile-comment-search
Feb 23, 2026
Merged

Fix profile comment search — wire up term param to API and backend query#3695
IanPhilips merged 1 commit intomainfrom
fix/profile-comment-search

Conversation

@todwadd
Copy link
Collaborator

@todwadd todwadd commented Feb 16, 2026

The search input on the profile Comments tab was completely non-functional: the frontend never passed the search term to the API, and the API/backend had no support for it. Added term param to the schema, ILIKE filtering on comment text/content and market question in the SQL query, and wired the frontend to pass the debounced search term.

The search input on the profile Comments tab was completely non-functional:
the frontend never passed the search term to the API, and the API/backend
had no support for it. Added term param to the schema, ILIKE filtering on
comment text/content and market question in the SQL query, and wired the
frontend to pass the debounced search term.
@vercel
Copy link

vercel bot commented Feb 16, 2026

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

Project Deployment Actions Updated (UTC)
dev Ready Ready Preview, Comment Feb 17, 2026 0:00am
prod Ready Ready Preview, Comment Feb 17, 2026 0:00am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Ignored Ignored Feb 17, 2026 0:00am

Request Review

@todwadd
Copy link
Collaborator Author

todwadd commented Feb 17, 2026

alternatively I can remove this search, but I assume someone put it in there intentionally

@todwadd todwadd requested a review from IanPhilips February 23, 2026 22:02
@IanPhilips IanPhilips merged commit 3df58ba into main Feb 23, 2026
6 checks passed
@IanPhilips IanPhilips deleted the fix/profile-comment-search branch February 23, 2026 22:11
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