Skip to content

Add browse definition handling in BrowseByDateComponent#5121

Open
xvThomas wants to merge 1 commit intoDSpace:mainfrom
xvThomas:xvThomas/issue5120
Open

Add browse definition handling in BrowseByDateComponent#5121
xvThomas wants to merge 1 commit intoDSpace:mainfrom
xvThomas:xvThomas/issue5120

Conversation

@xvThomas
Copy link

@xvThomas xvThomas commented Feb 17, 2026

Fixes #5122

@tdonohue
Copy link
Member

tdonohue commented Feb 17, 2026

@xvThomas : Thanks for this PR. Just a few notes:

  1. This PR has a merge conflict with out main branch. Could you clean that up please so that automated tests are able to run? You may need to rebase this PR against the latest main code, or similar.
  2. Also, it'd be good to provide details to others about how to test this PR. As all our reviewers/testers are volunteers, it can help to find volunteers if you can describe (in the PR description) the steps to take to verify that the PR is working properly. That's why we provide a PR template that we ask everyone to fill out. So, if you have time, I think it'd also be useful to add testing steps to the description.

@tdonohue tdonohue added bug component: Discovery related to discovery search or browse system 1 APPROVAL pull request only requires a single approval to merge labels Feb 17, 2026
@tdonohue tdonohue moved this to 🙋 Needs Reviewers Assigned in DSpace 10.0 Release Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1 APPROVAL pull request only requires a single approval to merge bug component: Discovery related to discovery search or browse system

Projects

Status: 🙋 Needs Reviewers Assigned

Development

Successfully merging this pull request may close these issues.

browse-by-date-component is using a wrong item metadata

2 participants