Skip to content

Make changes to migration and filtering to support default date value - #31

Merged
jloux-brapi merged 1 commit into
epic/BI-2862from
bug/BI-3066
Oct 8, 2026
Merged

jloux-brapi merged 1 commit into
epic/BI-2862from
bug/BI-3066

Conversation

@jloux-brapi

@jloux-brapi jloux-brapi commented Oct 6, 2026 •

Copy link
Copy Markdown

Description

Story: BI-3066

  • Migration modified to support default date value if additional_info.created_date = NULL
  • SearchQueryBuilder modified to support filtering ignoring the default date

Dependencies

Testing

  • Try filtering Germplasm values on 1970, which is contained in the default value
  • Verify unit tests are not failing because they are not supplying created_date

Checklist:

@jloux-brapi
jloux-brapi requested review from ahulsekemp and humsika and removed request for ahulsekemp October 6, 2026 21:44

@humsika humsika left a comment

Copy link
Copy Markdown

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 85c1116 into epic/BI-2862 Oct 8, 2026
1 check passed
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