Skip to content

Update term_browser and entry replacements in ohd.yml#1090

Merged
jamesaoverton merged 2 commits intomasterfrom
wdduncan-patch-1
Mar 3, 2026
Merged

Update term_browser and entry replacements in ohd.yml#1090
jamesaoverton merged 2 commits intomasterfrom
wdduncan-patch-1

Conversation

@wdduncan
Copy link
Copy Markdown
Member

@wdduncan wdduncan commented Mar 3, 2026

Changed redirect to OLS.

Comment thread config/ohd.yml Outdated
@jamesaoverton jamesaoverton merged commit 15009c2 into master Mar 3, 2026
1 of 2 checks passed
@jamesaoverton jamesaoverton deleted the wdduncan-patch-1 branch March 3, 2026 19:35
@wdduncan
Copy link
Copy Markdown
Member Author

wdduncan commented Mar 3, 2026

Thanks @jamesaoverton
What was the issue? Wondering what I did wrong.

@jamesaoverton
Copy link
Copy Markdown
Member

The /about/ entry needed two changes:

  1. It's better to use the /terms/ endpoint in OLS, because you also want to handle properties and not just /classes/. OLS will redirect to classes or properties as appropriate.
  2. The /terms/ endpoint also needs the first part of the IRI.

See my commit a311fa5.

@wdduncan
Copy link
Copy Markdown
Member Author

wdduncan commented Mar 3, 2026

Thanks!

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.

2 participants