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

Added lesson metadata and schema.org JSON-LD rendering #41

Closed
wants to merge 528 commits into from

Conversation

njall
Copy link

@njall njall commented Jul 22, 2016

Hi,

I thought it might be a good idea to help the discoverability of training materials by having more metadata examples in the config, and by rendering the metadata into a schema.org CreativeWork specification using the JSON-LD representation.

Structuring metadata using Schema.org is very useful for SEO to get included in more Google SERPs; as well as for content integration platforms like TeSS - The Life Science Training Portal. There's more about it on the working group page for BioSchemas

The variables declared in the _config.yml file are the attributes of the lesson correlated to the CreativeWork type. They are declared there and then rendered in JSON-LD in the schema_org.html partial in 'includes' folder. This is partial is called in _layouts/lesson.html

I've added a few example attributes but these variables could theoretically be the whole of http://schema.org/CreativeWork

@njall
Copy link
Author

njall commented Jul 22, 2016

See issue #40

@k8hertweck
Copy link
Contributor

This looks fine to me, and I'm happy to accommodate changes to this template that match with other lessons in our organization.

keywords: GitHub, Forking, Collaborative # see: https://schema.org/keywords
audience: [PostDoc students, Early Career Researchers] # see: https://schema.org/audience
license: "https://creativecommons.org/licenses/by/3.0/" # see: schema.org/license
author: [Homer Simpson, Ned Flanders] # see: https://schema.org/author
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can author be auto-generated from the Git commits and .mailmap? (That's how we generate author lists for publication purposes.)

rgaiacs and others added 21 commits November 19, 2017 21:05
If _extras/figures.md exists,

    $ ./bin/lesson_initialize.py

will return

    Warning: _extras/figures.md already exists.
    **Exiting without creating files.**
Changed the keyword for emailing from 'contact' to 'email' per carpentries/styles#178 and changed the description for how to use GitHub to reflect usage of the web interface.
Reorganized description of process for contributing to clarify/increase readability
Add note about version of Jekyll needed per carpentries/styles#195 (comment).
Add HTML indenting.
stijnvanhoey and others added 25 commits June 11, 2018 19:56
Makefile: add docker-serve target
Add links.md to files in boilerplate
update URLs to reflect consolidation to carpentries organization
* Add 'cp' (Carpentry) as a lesson type (for instructor training)
* Explain lesson type abbreviations (swc, dc, lc, cp)
* Emphasize the fact that there is no need to edit "repository" variable in front matter
@maxim-belkin
Copy link
Contributor

Could you please move your suggested changes to a different branch? It does not make sense to merge 250 commits that already exist in the base (gh-pages) branch.

I guess your commit is eb8da64, so the following set of commands will do it:

git reset --hard origin/gh-pages
git checkout -b my-changes
git cherry-pick eb8da64
git push https://github.com/carpentries/lesson-example

You can also enable edits from maintainers for this PR - I will be able to help you with that then.

@rgaiacs
Copy link
Contributor

rgaiacs commented Jul 31, 2018

Could you please move your suggested changes to a different branch?

@njall works with me. I'm taking over this pull request to work during this week.

@rgaiacs
Copy link
Contributor

rgaiacs commented Aug 2, 2018

@njall Thanks very much for the pull request. I'm taking it over from #226.

@rgaiacs rgaiacs closed this Aug 2, 2018
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.