Lee-W commented on code in PR #70498:
URL: https://github.com/apache/airflow/pull/70498#discussion_r3680417862
##########
registry/src/js/provider-filters.js:
##########
@@ -40,9 +40,11 @@
const lifecycle = item.dataset.lifecycle;
const name = item.dataset.name || '';
const categories = item.dataset.categories || '';
+ const integrations = item.dataset.integrations || '';
const matchesLifecycle = currentLifecycle === 'all' || lifecycle ===
currentLifecycle;
- const matchesSearch = name.includes(currentSearch.toLowerCase());
+ const search = currentSearch.toLowerCase();
+ const matchesSearch = name.includes(search) ||
integrations.includes(search);
Review Comment:
Both sides collapse the same way now. `normalize()` is duplicated into the
IIFE with a comment pointing back at `_data/providerKeywordMatch.js`.
I swept every provider name and integration name as a query against the
rendered attributes: no existing match is lost, and `pydantic-ai`,
`pydantic_ai`, `mcp-server`, `common-ai` and `speech to text` all resolve now.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
