Fix spurious logging triggered from Trello searches.

Review Request #14185 — Created Sept. 25, 2024 and submitted — Latest diff uploaded

Information

rbintegrations
release-4.x

Reviewers

While testing out the Trello integration, I noticed that if I typed in a
search and then backspaced it out, we'd get errors in the log about
invalid API requests. The problem is that our selectize-based search
triggers an API request every time it changes, even when the input is
emptied out, but the Trello API requires the search text to be at least
1 character long.

This change makes it so if the search query is empty, we just directly
return an empty list to the client instead of trying to make an API
request to Trello.

Emptied out my search input and saw that the search results drop-down
disappeared without any ugly errors being logged on the server.

Commits

Files