Skip to content

Add OAI-PMH subscribe button#1317

Merged
fbacall merged 2 commits into
ElixirTeSS:masterfrom
pan-training:copy_oai_pmh_button
Jun 5, 2026
Merged

Add OAI-PMH subscribe button#1317
fbacall merged 2 commits into
ElixirTeSS:masterfrom
pan-training:copy_oai_pmh_button

Conversation

@eilmiv
Copy link
Copy Markdown
Collaborator

@eilmiv eilmiv commented Jun 3, 2026

Summary of changes

  • Added OAI-PMH subscribe button and modal with minimal explanation and OAI-PMH link

Motivation and context

Closes #1316

Screenshots

image

Checklist

  • I have read and followed the CONTRIBUTING guide.
  • I confirm that I have the authority necessary to make this contribution on behalf of its copyright owner and agree to license it to the TeSS codebase under the BSD license.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds an OAI-PMH discovery/copy flow to the existing “Subscribe” UI, making the instance’s OAI-PMH endpoint easier to find and copy (per #1316).

Changes:

  • Adds a new “Harvest using OAI-PMH” entry to the Subscribe dropdown for Materials.
  • Introduces a Bootstrap modal that displays the OAI-PMH endpoint with a clipboard-copy button and brief guidance.
  • Adds English locale strings for the new UI label and modal explanation text.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
config/locales/en.yml Adds new i18n strings for the OAI-PMH subscribe entry and modal instructions.
app/views/subscriptions/_subscribe_to_oai_pmh_modal.html.erb New modal partial that displays the OAI-PMH endpoint and copy-to-clipboard button.
app/views/subscriptions/_subscribe_button.html.erb Adds the new OAI-PMH option to the Subscribe dropdown and renders the new modal for Materials.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread config/locales/en.yml
@eilmiv eilmiv marked this pull request as ready for review June 3, 2026 12:32
@eilmiv eilmiv requested a review from fbacall June 3, 2026 12:32
@fbacall fbacall merged commit 3405882 into ElixirTeSS:master Jun 5, 2026
7 checks passed
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.

Add a button to copy OAI-PMH link

3 participants