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

improve URL rendering on MESH pages #119

Open
VladimirAlexiev opened this issue Feb 25, 2019 · 2 comments
Open

improve URL rendering on MESH pages #119

VladimirAlexiev opened this issue Feb 25, 2019 · 2 comments

Comments

@VladimirAlexiev
Copy link

A MESH page eg https://id.nlm.nih.gov/mesh/D008071.html uses ugly URLs, eg

Given that all these URLs resolve, please present them in their canonic form:

Otherwise it becomes unnecessarily complex to share MESH semantic URLs.

There are also links over ontology terms, eg

Because these use hash, presenting them in their canonic form

will be a problem because the browser will send only https://id.nlm.nih.gov/mesh/vocab, resulting in the whole ontology being returned.

Perhaps you can use the canonic form in the link name, but the describe form in the link, eg

<a href="https://id.nlm.nih.gov/mesh/describe?uri=http%3A%2F%2Fid.nlm.nih.gov%2Fmesh%2Fvocab%23TopicalDescriptor">
https://id.nlm.nih.gov/mesh/vocab#TopicalDescriptor
</a>
@danizen
Copy link
Contributor

danizen commented Feb 25, 2019

@VladimirAlexiev, in principal, a "lodestar" should be able to host IRIs that may not be permanently resolvable or which may not be hosted on that particular server. Using canonical forms can lead to odd results occasionally, for instance, try this query:

SELECT DISTINCT ?g
WHERE {
  GRAPH ?g { ?s ?p ?o }
} 

On the other hand, I caused this issue so that development on http://localhost:8080/ would refer to the development system rather than to the production installation. That is a basic requirement for my automated testing of the system, but I think it could be done in a different way where a REST URI beginning with https://id.nlm.nih.gov/ is rewritten to a baseURI.

@VladimirAlexiev
Copy link
Author

Perhaps you can patch lodestar to do this better thing for MESH URLs?

danizen pushed a commit that referenced this issue Oct 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants