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

Links from the description field are not valid URLs #852

Open
1 of 2 tasks
pjmonks opened this issue Mar 27, 2024 · 4 comments
Open
1 of 2 tasks

Links from the description field are not valid URLs #852

pjmonks opened this issue Mar 27, 2024 · 4 comments
Assignees
Labels
bug Something isn't working epic An issue representing a collection of other issues

Comments

@pjmonks
Copy link
Contributor

pjmonks commented Mar 27, 2024

Description

When testing on the NHS Data Dictionary instance - https://modelcatalogue.cs.ox.ac.uk/nhsd-datadictionary/ - the hyperlinks in the description fields of an item are just the Mauro path, but are not valid URLs to redirect to.

Steps to reproduce

Go to an item, like this: https://modelcatalogue.cs.ox.ac.uk/nhsd-datadictionary/#/catalogue/dataClass/6d054bbe-e634-45ee-837a-ed7f83c3f77c//15deb5ca-e327-4f5a-b240-eedfdd5e3fd3/description

Notice that the description field contains what looks like hyperlinks:

image

However, copy the link address from any of these links e.g. "ACCOMODATION", and this is the exact href from the <a> tag:

dm:Classes and Attributes|dc:Classes|dc:ACCOMMODATION

Click on these links, and the browser will not know what to do.

Expected behavior

These hyperlinks should be valid URLs to redirect to.

Related Issues

The fix for this is related to these issues:

@pjmonks pjmonks added the bug Something isn't working label Mar 27, 2024
@pjmonks
Copy link
Contributor Author

pjmonks commented Mar 27, 2024

I feel like I have worked on these links previous (perhaps with the Markdown editor, not the HTML version) - there may be existing PRs in this repo related to updates I worked on for hyperlinks, but we will need to locate them.

@jamesrwelch
Copy link
Contributor

We certainly did some work in this space - it may be that the ingest / publication needs aligning with whatever works through the UI. Definitely an issue to get sorted though

@pjmonks
Copy link
Contributor Author

pjmonks commented Apr 23, 2024

My initial testing shows that creating a hyperlink to a Mauro item manually through the UI works fine and generates a valid URL reference. The next step would be to inspect the DD ingest code like @jamesrwelch suggested, this is probably where the incorrect URLs are being generated.

@pjmonks
Copy link
Contributor Author

pjmonks commented Apr 30, 2024

Based on feedback about the ingest of the DD and links, I've created:

This will keep links as paths but allow them to be displayed correctly.

@pjmonks pjmonks added the epic An issue representing a collection of other issues label May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working epic An issue representing a collection of other issues
Projects
None yet
Development

No branches or pull requests

2 participants