Skip to content

Conversation

ekraffmiller
Copy link
Contributor

What this PR does / why we need it:
Adds json-ld to header of View Dataset page.

Which issue(s) this PR closes:

Closes #350

Special notes for your reviewer:
This PR is a proof of concept, to test that the json-ld is available in a deployed app. The json-ld is viewable in the browser view of the DOM, and has dynamic data based on the current dataset, but we need to test on a webpage that can be run in https://search.google.com/test/rich-results

Suggestions on how to test this:
Deploy to beta.dataverse.org and do the rich-results test.

Does this PR introduce a user interface change? If mockups are available, please link/include them here:
no
Is there a release notes update needed for this change?:
no
Additional documentation:

@coveralls
Copy link

coveralls commented May 24, 2024

Coverage Status

coverage: 98.403% (+0.6%) from 97.805%
when pulling 1611065 on feature/350-json-ld
into 0d1d97e on develop.

@pdurbin
Copy link
Member

pdurbin commented May 24, 2024

Deploy to beta.dataverse.org and do the rich-results test.

It would be nice for beta to only run merged code but whatever works for now!

Maybe we need an alpha server. 😄

@ekraffmiller
Copy link
Contributor Author

Deploy to beta.dataverse.org and do the rich-results test.

It would be nice for beta to only run merged code but whatever works for now!

Maybe we need an alpha server. 😄

@pdurbin yes, I agree we don't want to do this generally, I just wanted to deploy this to beta so that we could run google rich results, to test that the json-ld is readable in the page. We did the test, and found out that the containerized dataverse image runs with robots.txt disallowing all access. So that is the next thing to figure out for this ticket :)

@ekraffmiller
Copy link
Contributor Author

I created a Dataverse issue for customizing robots.txt: IQSS/dataverse#10593

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.

Add Schema.org or Croissant metadata to header of Dataset view page

4 participants