Skip to content

105 restore request this collection button for collections with no contents - #106

Merged
Nealium104 merged 2 commits into
mainfrom
105-restore-request-this-collection-button-for-collections-with-no-contents
Sep 24, 2026
Merged

Nealium104 merged 2 commits into
mainfrom
105-restore-request-this-collection-button-for-collections-with-no-contents

Conversation

@Nealium104

Copy link
Copy Markdown
Contributor

This PR restores the ability to request UK collections that have no contents listed. These previously had a "Request This Collection" button, but that behavior was removed during limestone-rewrite work.

Changes

  • Adds an "Add Collection to Request" button to the Requests box for UK collections with no contents
  • Keeps the button outside the request dropdown so it is always visible
  • Uses the existing request form for retrievals, reproductions, and saving for later
  • Adds the checkered divider between the controls when the panel is open

Fixes #105

Screenshots of implementation

Screenshot 2026-09-23 150003 Screenshot 2026-09-23 150012

@Nealium104
Nealium104 requested a review from a team as a code owner September 23, 2026 19:01

@cokernel cokernel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Verified functionality.

Verified that the expected request_options shows up on requests-libraries when requesting a collection.

This looks good to me.

@Nealium104
Nealium104 merged commit ea81f39 into main Sep 24, 2026
3 checks passed
@Nealium104
Nealium104 deleted the 105-restore-request-this-collection-button-for-collections-with-no-contents branch September 24, 2026 13:39
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.

Restore "Request This Collection" button for collections with no contents

2 participants