Skip to content

UILD-748: Fix the relation URLs to match with the URLs defined in https://bibfra.me/vocab/relation#334

Merged
pkjacob merged 1 commit intomasterfrom
pjacob/UILD-748
Mar 12, 2026
Merged

UILD-748: Fix the relation URLs to match with the URLs defined in https://bibfra.me/vocab/relation#334
pkjacob merged 1 commit intomasterfrom
pjacob/UILD-748

Conversation

@pkjacob
Copy link
Contributor

@pkjacob pkjacob commented Mar 11, 2026

No description provided.

@github-actions
Copy link

github-actions bot commented Mar 11, 2026

Jest Unit Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit 1caedb8. ± Comparison against base commit 5e945ba.

♻️ This comment has been updated with latest results.

Copy link

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

Updates the BIBFRAME contribution relation URI mappings to align with the canonical relation URIs defined by https://bibfra.me/vocab/relation, ensuring generated records use the correct predicate URLs.

Changes:

  • Adjust _contributions keys in SIMPLE_LOOKUP_MAPPING to use the correct bibfra.me relation URL forms (e.g., camelCase terms instead of abbreviations/legacy forms).
  • Add a changelog entry and Jira link reference for UILD-748.

Reviewed changes

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

File Description
src/common/constants/bibframeMapping.constants.ts Updates contribution relation mapping keys to match bibfra.me relation vocabulary URLs.
CHANGELOG.md Documents the change and adds the UILD-748 reference link.

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

@sonarqubecloud
Copy link

@pkjacob pkjacob merged commit 2eadfe7 into master Mar 12, 2026
15 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.

4 participants