Skip to content

chore: allow undefined for autocomplete limits (v2.8.1)#221

Merged
k-gerner merged 1 commit intomainfrom
allow-undefined-payload-autocomplete-limit-2.8.1
Jan 7, 2026
Merged

chore: allow undefined for autocomplete limits (v2.8.1)#221
k-gerner merged 1 commit intomainfrom
allow-undefined-payload-autocomplete-limit-2.8.1

Conversation

@k-gerner
Copy link
Copy Markdown
Contributor

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

This will allow us to "reset" the value, so that we don't pass the param in the request. Note that this currently doesn't "error" per say, but there could be a type warning, so may as well just clean it up.

@k-gerner k-gerner requested a review from a team as a code owner January 7, 2026 22:07
@k-gerner k-gerner changed the title chore: allow undefined for autocomplete limits chore: allow undefined for autocomplete limits (v2.8.1) Jan 7, 2026
@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 20798092126

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 91.358%

Totals Coverage Status
Change from base Build 20792356930: 0.0%
Covered Lines: 381
Relevant Lines: 408

💛 - Coveralls

Copy link
Copy Markdown
Contributor

@Fondryext Fondryext left a comment

Choose a reason for hiding this comment

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

Just to check we don't want to edit src/search-headless.ts's calls?

@k-gerner k-gerner merged commit a00ee46 into main Jan 7, 2026
11 checks passed
@k-gerner k-gerner deleted the allow-undefined-payload-autocomplete-limit-2.8.1 branch January 7, 2026 22:12
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