Skip to content

feat: pass autocomplete limit if present (v2.8.0)#324

Merged
k-gerner merged 4 commits intomasterfrom
autcomplete-limit-2.8.0
Jan 6, 2026
Merged

feat: pass autocomplete limit if present (v2.8.0)#324
k-gerner merged 4 commits intomasterfrom
autcomplete-limit-2.8.0

Conversation

@k-gerner
Copy link
Contributor

@k-gerner k-gerner commented Jan 6, 2026

If the limit param is present on autocomplete requests, pass include it in the search API request. Support for limit in /autocomplete was recently added to the backend Search API.

J=WAT-5263
TEST=manual
Made necessary changes to answers-search-ui and answers-hitchhiker-theme and saw the requests in the network tab correctly populated w/ the limit param

@k-gerner k-gerner requested a review from a team as a code owner January 6, 2026 20:15
@coveralls
Copy link

coveralls commented Jan 6, 2026

Pull Request Test Coverage Report for Build 20761167724

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 89.397%

Totals Coverage Status
Change from base Build 18479230065: 0.0%
Covered Lines: 244
Relevant Lines: 266

💛 - Coveralls

Fondryext
Fondryext previously approved these changes Jan 6, 2026
@k-gerner k-gerner merged commit a29b0af into master Jan 6, 2026
15 checks passed
@k-gerner k-gerner deleted the autcomplete-limit-2.8.0 branch January 6, 2026 20:59
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.

3 participants