Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update duo.yml #479

Merged
merged 3 commits into from Jan 29, 2019
Merged

Update duo.yml #479

merged 3 commits into from Jan 29, 2019

Conversation

mcourtot
Copy link
Contributor

Added home
Updated releases and about link to point to the releases page on GH and to the OLS

@jamesaoverton the fall through doesn't seem to work properly can you help? I didn't update this and was hoping that using http://purl.obolibrary.org/obo/duo/ would redirect to the GH repo by default but it instead points to berkeley

Added home
Updated releases and about link to point to the releases page on GH and to the OLS

@jamesaoverton the fall through doesn't seem to work properly can you help? I didn't update this and was hoping that using http://purl.obolibrary.org/obo/duo/ would redirect to the GH repo by default but it instead points to berkeley
@jamesaoverton
Copy link
Member

I'm seeing http://purl.obolibrary.org/obo/duo/ redirect to https://raw.githubusercontent.com/EBISPOT/DUO/master/. I'm not sure what you want it to do.

If you drop the trailing slash, then http://purl.obolibrary.org/obo/duo goes to http://ontologies.berkeleybop.org/, but that's a top-level PURL that you're not supposed to use, and it's different issue about top-level failovers #251.

This PR looks fine, but doesn't change that prefix: / rule.

@matentzn matentzn merged commit 39aacb0 into master Jan 29, 2019
@matentzn matentzn deleted the mcourtot-patch-6 branch January 29, 2019 12:38
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.

None yet

3 participants