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

6 add schemaorg data to head of view page #15

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

nbodony
Copy link
Collaborator

@nbodony nbodony commented Aug 26, 2024

Json-ld string is placed in head of view.html and can be found at /ld/dataset. It only includes the information that I was aware of and thought to include so it's definitely not complete but I think it works how it's supposed to and is placed everywhere it needs to be.

/ld/piglife:
image

/ld/slamdataset: (the list of authors is very long)
image

/view/slamdataset:
image

/view/piglife
image

creates an initial and very incomplete schema string which can reached at /ld/dataset
@nbodony nbodony linked an issue Aug 26, 2024 that may be closed by this pull request
@nbodony nbodony marked this pull request as draft August 26, 2024 03:05
make schema a python dict
convert dict to json string
make variable list of creator blocks
move script to view.html
pass schema_string to view.html head
@nbodony nbodony changed the title 6 add schemaorg data to head of view page (super duper not done yet) 6 add schemaorg data to head of view page Sep 5, 2024
@nbodony nbodony marked this pull request as ready for review September 5, 2024 03:12
@nbodony nbodony requested a review from robkooper September 5, 2024 03:17
Removed unnecessary arrays and added one necessary array

added keywords with correct spacing after colon

commented out unused lines in json_ld_dict

replaced 'schema' with 'json_ld'

added missing spaces after colons
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 data to head of view page
2 participants