Skip to content

DRYD-2073: Object Record > All Profiles > Add Home Location#541

Open
spirosdi wants to merge 3 commits into
developfrom
feature/dryd-2073
Open

DRYD-2073: Object Record > All Profiles > Add Home Location#541
spirosdi wants to merge 3 commits into
developfrom
feature/dryd-2073

Conversation

@spirosdi

@spirosdi spirosdi commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

What does this do?
It adds a repeatable homeLocationGroupList to the collectionobject common schema. So that it is available in advanced search results and it can be displayed alongside the computed current location.

Why are we doing this? (with JIRA link)
We need to add a repeating group of homeLocation and homeLocationNote fields along the computedCurrentLocation field. We also need to display all the home locations in the advanced search results. https://collectionspace.atlassian.net/browse/DRYD-2073

How should this be tested? Do these changes have associated tests?
Please follow the steps in the PR: collectionspace/cspace-ui.js#355

Dependencies for merging? Releasing to production?
The following PRs should be released along:
collectionspace/application#352
collectionspace/cspace-ui.js#355
collectionspace/cspace-ui-plugin-profile-anthro.js#44
collectionspace/cspace-ui-plugin-profile-fcart.js#33
collectionspace/cspace-ui-plugin-profile-lhmc.js#34
collectionspace/cspace-ui-plugin-profile-materials.js#22
collectionspace/cspace-ui-plugin-profile-publicart.js#35

Has the application documentation been updated for these changes?
The changelog has been updated.

Did someone actually run this code to verify it works?
@spirosdi ran it locally

Have any new security vulnerabilities been handled?
no new vulnerabilities introduced

@mikejritter mikejritter 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.

Tested locally and verified objects with

  • No home location
  • Single home location
  • Multiple home locations

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