Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SearchControls: Validate filter columns more efficiently #83

Merged
merged 2 commits into from
Jun 9, 2022

Conversation

nilmerg
Copy link
Member

@nilmerg nilmerg commented Jun 8, 2022

Greatly enhances what users can type into the search bar now. Previously some relation columns didn't successfully validate in the search bar. Now any relation column can be successfully used.

Includes a single bc: SearchControls::fetchMetaData is now called fetchFilterColumns (The return value hasn't changed)

requires Icinga/ipl-orm#64

@nilmerg nilmerg added the enhancement New feature or request label Jun 8, 2022
@nilmerg nilmerg added this to the v0.5.0 milestone Jun 8, 2022
@nilmerg nilmerg self-assigned this Jun 8, 2022
@cla-bot cla-bot bot added the cla/signed label Jun 8, 2022
@nilmerg nilmerg force-pushed the enhance-filter-validation branch from b4a4b90 to 82488fc Compare June 9, 2022 09:18
@nilmerg nilmerg merged commit b276737 into master Jun 9, 2022
@nilmerg nilmerg deleted the enhance-filter-validation branch June 9, 2022 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla/signed enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant