Skip to content

add support for collections#220

Merged
rshewitt merged 6 commits intomainfrom
get-children-from-parent
Apr 10, 2026
Merged

add support for collections#220
rshewitt merged 6 commits intomainfrom
get-children-from-parent

Conversation

@rshewitt
Copy link
Copy Markdown
Contributor

@rshewitt rshewitt commented Mar 31, 2026

related to 5653

adds support for collections

  • this isn't a separate route but a new query parameter similar to current catalog on the index/main search page
  • adds a collection card pinned to top of search page
  • this expects the parent/child relationship to be established. this doesn't calculate the relationship for dcatus.
  • this updates the opensearch mapping object so a complete re-index will be required after merge

update

  • example collection on dev. apparently rebuilding the index didn't catch all the documents? i confirmed the dataset(s) were in the catalog read-only db before recreating it but running the sync compare after some time did the charm.

@rshewitt rshewitt requested a review from a team March 31, 2026 17:43
@rshewitt rshewitt changed the title Get children from parent add support for collections Mar 31, 2026
@rshewitt rshewitt marked this pull request as draft March 31, 2026 21:31
@rshewitt rshewitt marked this pull request as ready for review March 31, 2026 22:54
@rshewitt rshewitt marked this pull request as draft April 2, 2026 15:28
@rshewitt rshewitt force-pushed the get-children-from-parent branch from 3283146 to 7af7558 Compare April 6, 2026 17:00
@rshewitt rshewitt marked this pull request as ready for review April 6, 2026 17:02
cmhedrickREI
cmhedrickREI previously approved these changes Apr 10, 2026
Copy link
Copy Markdown
Member

@cmhedrickREI cmhedrickREI left a comment

Choose a reason for hiding this comment

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

Not blocking:
I would like to see maybe a bit of padding or margin to help give space to the stats at the bottom of the card. But that's a personal preference.

Image

This was some great work 👍

Copy link
Copy Markdown
Member

@cmhedrickREI cmhedrickREI left a comment

Choose a reason for hiding this comment

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

Thank you for the update 🙌

@rshewitt rshewitt merged commit 1b930b1 into main Apr 10, 2026
6 checks passed
@rshewitt rshewitt deleted the get-children-from-parent branch April 10, 2026 16:14
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