Skip to content

Add numeric range filtering for dashboard and Biobank filters#10501

Open
Montekkundan wants to merge 2 commits into
aces:mainfrom
Montekkundan:montek/8939-numeric-range-filters
Open

Add numeric range filtering for dashboard and Biobank filters#10501
Montekkundan wants to merge 2 commits into
aces:mainfrom
Montekkundan:montek/8939-numeric-range-filters

Conversation

@Montekkundan
Copy link
Copy Markdown
Contributor

Brief summary of changes

  • Adds a reusable NumericRangeElement form component for min/max numeric filtering.
  • Extends shared table filtering and filter query-string handling to support <field>Min / <field>Max range values.
  • Uses the numeric range filter for Biobank Age at Collection.
  • Adds a Candidate Age at Registration range filter to Statistics recruitment dashboard charts and applies it in the chart API with parameterized age bounds.

Testing instructions (if applicable)

  • On the Statistics dashboard, open Recruitment filters, enter a Candidate Age range such as 20 to 40, and confirm chart requests include candidateAgeMin=20&candidateAgeMax=40.
  • Open http://localhost:8080/biobank/?ageMin=10&ageMax=30 and confirm Age at Collection shows Minimum/Maximum numeric range inputs populated from the URL.
  • Run php -l modules/statistics/php/charts.class.inc and targeted ESLint on the changed JS/JSX files.

Link(s) to related issue(s)

@github-actions github-actions Bot added Language: PHP PR or issue that update PHP code Language: Javascript PR or issue that update Javascript code Module: statistics PR or issue related to statistics module labels May 16, 2026
@ridz1208 ridz1208 requested a review from HenriRabalais May 18, 2026 23:53
Comment thread jsx/Filter.js Outdated
Comment thread modules/statistics/jsx/widgets/helpers/queryChartForm.js Outdated
Copy link
Copy Markdown
Collaborator

@HenriRabalais HenriRabalais left a comment

Choose a reason for hiding this comment

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

lgtm!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language: Javascript PR or issue that update Javascript code Language: PHP PR or issue that update PHP code Module: statistics PR or issue related to statistics module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[JSX] Add range capabilities to the filters

2 participants