JCU/Add CitacePRO configuration (ported from customer/mendelu)#1361
JCU/Add CitacePRO configuration (ported from customer/mendelu)#1361Kasinhou wants to merge 2 commits into
Conversation
Adds the CitacePRO citation-service configuration required by the frontend item-page citation widget: - dspace.cfg: citace.pro.url, citace.pro.university, citace.pro.allowed. citace.pro.university is a FLAGGED PLACEHOLDER (dspace.jcu.cz) -- the real JCU CitacePRO-registered repository host must be confirmed with the customer. - modules/rest.cfg: expose the three citace.pro.* properties via REST so the UI can read them through ConfigurationDataService. Ported from dataquest-dev/DSpace #1210. The OAI-PMH oai_dc.xsl citation formatting (#1233 / #1297 / #1312 / #1314) is intentionally NOT included: it is metadata-schema-specific to mendelu (local.volume, local.number, dc.relation.ispartof, dc.description.version) and will be applied separately on JCU prod against JCU's own metadata schema. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Ports CitacePRO backend configuration into the JCU customer branch so the frontend item-page widget can read required settings via REST.
Changes:
- Added CitacePRO configuration properties (
citace.pro.url,citace.pro.university,citace.pro.allowed) todspace.cfg. - Exposed the CitacePRO properties via REST by adding them to
rest.properties.exposedinmodules/rest.cfg.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| dspace/config/dspace.cfg | Adds CitacePRO config values used by the frontend widget. |
| dspace/config/modules/rest.cfg | Exposes CitacePRO config properties over REST for UI consumption. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
What is CitacePRO?
CitacePRO is a Czech citation-generation service. The DSpace frontend embeds a CitacePRO
<iframe>on the item page; CitacePRO composes the citation from metadata it harvests via OAI‑PMH from the institution's public repository.This PR ports the CitacePRO configuration from
customer/mendelutocustomer/jcu, adapted for JCU. It is the backend half of the CitacePRO item‑page widget (companion frontend PR: dataquest-dev/dspace-angular, branchfeature/jcu-citacepro).What was ported
dspace/config/dspace.cfg— newCitacepro configblock:citace.pro.url = https://www.citacepro.com/api/dspace/citace/oai(shared CitacePRO API base)citace.pro.university = dspace.jcu.cz—citace.pro.allowed = truedspace/config/modules/rest.cfg— exposes the three properties over REST so the UI can read them viaConfigurationDataService:rest.properties.exposed = citace.pro.url/citace.pro.allowed/citace.pro.universitySource (customer/mendelu)
Branch: https://github.com/dataquest-dev/DSpace/tree/customer/mendelu
dspace.cfg+modules/rest.cfg) — ported.Intentionally NOT ported (per decision with the requester)
The mendelu OAI‑PMH work that shapes
oai_dc.xslfor CitacePRO harvesting:Reasons:
local.volume,local.number,dc.relation.ispartof,dc.description.version,dc.identifier.isbn) and would produce wrong output if applied blindly to a different schema.Also skipped: the old ZCU PR #686 (put the same props in
clarin-dspace.cfg; superseded by #1210 which relocated them todspace.cfg), and the closed duplicate #1311 (superseded by #1312).citace.pro.university = dspace.jcu.czis a placeholder. The real value is JCU's CitacePRO-registered repository host (mendelu usesrepozitar.mendelu.cz). It must be confirmed with the customer / CitacePRO and swapped in before production; the property is flagged with aTODO_JCUcomment indspace.cfg.Verification
GET /server/api/config/properties/citace.pro.allowed→["true"]GET /server/api/config/properties/citace.pro.university→["dspace.jcu.cz"]GET /server/api/config/properties/citace.pro.url→["https://www.citacepro.com/api/dspace/citace/oai"]