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

Moving to the new way of tagging tools in content #1249

Merged
merged 233 commits into from
Aug 18, 2023

Conversation

bedroesb
Copy link
Member

@bedroesb bedroesb commented May 22, 2023

Tools will be described in the main tools and resources yaml file:

image

And be linked from the content using

{% tool "tool-id" %}

This will be visualized in the content like this:

image

Still on the TODO:

  • Replace tool links in the text with the new snippet
  • Add missing tools to the main yaml file
  • https://github.com/bedroesb/rdmkit/blob/demo/_data/tool_and_resource_list.yml replace
  • Adapt content so tools get mentioned at least once @ rdmkit
  • Agree on how to handle country specific tools
  • in national pages, change ref_to_main_resources so that it makes use of ID
  • update instance_of to make use of the tool ID
  • agree on (not) having backwards compatibility for old way of having related pages at tools/spreaadsheet
  • Update to spreadsheet convertor script to work with the yaml file so the yaml file gets updated with FAIRsharing/TeSS and Bio.tools IDs automatically
  • Nicer/better dropdowns for the tool information
  • Make the tools in tools table alphabetically.
  • Add new way of tagging the tools to the ELIXIR Toolkit Theme
  • Update documentation in how to add a new tool or resource
  • Make the snippet more forgiving
  • Adding a linter to check for:
    • id is kebab-case
    • Description does not contain colon
    • URL works
    • Make sure that id linked in "instance_of" exists in the main yaml file

@bedroesb bedroesb requested a review from smza as a code owner August 16, 2023 11:25
Copy link
Member

@martin-nc martin-nc left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@bedroesb
Copy link
Member Author

This is a big moment, lets gooo 🚀

@bedroesb bedroesb merged commit 5cc09d9 into elixir-europe:master Aug 18, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contentathon Contentathon related issues and PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update request Plant sciences page ELIXIR-AAI -> LS Login