Skip to content

DRYD-2042: Update AdvancedSearch Response for Materials and Herbarium#508

Merged
mikejritter merged 1 commit intocollectionspace:developfrom
mikejritter:bug/dryd-2042-as-profile-fields
Mar 2, 2026
Merged

DRYD-2042: Update AdvancedSearch Response for Materials and Herbarium#508
mikejritter merged 1 commit intocollectionspace:developfrom
mikejritter:bug/dryd-2042-as-profile-fields

Conversation

@mikejritter
Copy link
Contributor

What does this do?

  • Add material to advanced search response
  • Add determinationTaxon to advanced search response

Why are we doing this? (with JIRA link)
Jira: https://collectionspace.atlassian.net/browse/DRYD-2042

When testing the second release candidate we saw that the material field was not being populated in the materials profile. This was due to us not checking if profiles had added any additional fields to the response on /collectionobject being used to populate the old table.

How should this be tested? Do these changes have associated tests?

  • Enable the materials and herbarium profiles
  • Rebuild collectionspace with this PR
  • In the materials profile plugin:
    • Create a CollectionObject with a material
    • Run a search on Objects
    • In the table view, see that the 'Material' column is populated
  • In the herbarium profile plugin:
    • Create a CollectionObject with a determination history taxon
    • Run a search on Objects
    • In the table view, see that the 'Scientific Name' column is populated

Dependencies for merging? Releasing to production?
None

Has the application documentation been updated for these changes?
The wiki for the advanced search response needs to be updated

Did someone actually run this code to verify it works?
@mikejritter tested locally

@mikejritter mikejritter requested a review from spirosdi February 27, 2026 22:08
Copy link
Contributor

@spirosdi spirosdi left a comment

Choose a reason for hiding this comment

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

Looks good, changes tested and work as expected.

@mikejritter mikejritter merged commit 04767c3 into collectionspace:develop Mar 2, 2026
@mikejritter mikejritter deleted the bug/dryd-2042-as-profile-fields branch March 2, 2026 17:13
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