Skip to content

Disable searching and filtering on Entry Number in Germplasm Lists - #498

Merged
jloux-brapi merged 1 commit into
feature/BI-2905from
feature/BI-3060
Sep 28, 2026
Merged

jloux-brapi merged 1 commit into
feature/BI-2905from
feature/BI-3060

Conversation

@jloux-brapi

@jloux-brapi jloux-brapi commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Description

Story: BI-3060

Disabled searching/filtering on Entry Number

Dependencies

None

Testing

  • When I load/refresh a Germplasm Lists details page, the page appears sorted on Entry Number to start, giving the sense that these are ordered in the order they were uploaded

  • When I hover over the Entry Number column, I expect there to not be an option to click to sort on this column. This functionality can be verified on the Pedigree column as well, where sorting is disabled there.

  • Under the Entry Number column, I expect the filter text box to not appear anymore like the other columns for filtering

  • All other columns and functionality on this page should work the same.

Checklist:

  • I have performed a self-review of my own code
  • I have tested my code and ensured it meets the acceptance criteria of the story
  • I have commented my code, particularly in hard-to-understand areas
  • I have either updated the source of truth or arranged for update with product owner if needed https://breedinginsight.atlassian.net/wiki/spaces/BI/pages/1559953409/Source+of+Truth
  • I have run SiteImprove on pages impacted by changes

@humsika humsika left a comment

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.

Code looks good to me.

@jloux-brapi
jloux-brapi merged commit c9f663f into feature/BI-2905 Sep 28, 2026
2 checks passed
@jloux-brapi
jloux-brapi deleted the feature/BI-3060 branch September 28, 2026 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants