Skip to content

Grid: suggest filters from grid search - #8191

Merged
labkey-nicka merged 24 commits into
developfrom
fb_grid_search
Oct 25, 2026
Merged

labkey-nicka merged 24 commits into
developfrom
fb_grid_search

Conversation

@labkey-nicka

@labkey-nicka labkey-nicka commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Rationale

This updates the grid search to offer and apply suggested filters based on search term.

Related Pull Requests

Changes

  • Build suggestions from cached per-column facts, ranked in one place
  • Let modules add column facts via providers, e.g. sample name expressions
  • Invalidate cached facts by data change token, else a short expiry
  • Share facts across users only when safe; never read PHI or audit-logged columns
  • Load facts in the background; requests wait at most 250 ms
  • Read facts through the grid's own table, scope and visible columns

Comment thread experiment/src/org/labkey/experiment/api/ExpMaterialTableImpl.java Outdated
Comment thread query/src/org/labkey/query/suggestions/ColumnFactsLoader.java
Comment thread query/src/org/labkey/query/suggestions/ColumnFactsLoader.java Outdated

private static @Nullable List<String> getValidValues(ColumnInfo column)
{
IPropertyValidator validator = PropertyService.get().getValidatorForColumn(column, PropertyValidatorType.TextChoice);

@XingY XingY Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If a field is text choice or multi choice, but didn't get to define valid values yet, will validator be null?

Comment thread query/src/org/labkey/query/suggestions/SuggestionTerm.java Outdated
@labkey-nicka
labkey-nicka force-pushed the fb_grid_search branch 2 times, most recently from 23048bd to b82c6d9 Compare October 23, 2026 23:25
@labkey-nicka
labkey-nicka merged commit 1a7f23f into develop Oct 25, 2026
8 of 12 checks passed
@labkey-nicka
labkey-nicka deleted the fb_grid_search branch October 25, 2026 19:48
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.

2 participants