Skip to content

Move to new way of tagging tools - #214

Merged
bedroesb merged 27 commits into
elixir-europe:mainfrom
bedroesb:new-way-tag-tools
Aug 24, 2023
Merged

Move to new way of tagging tools#214
bedroesb merged 27 commits into
elixir-europe:mainfrom
bedroesb:new-way-tag-tools

Conversation

@bedroesb

Copy link
Copy Markdown
Member

This pull request will close #198, progress tracking will be done on that issue.

Follow the transition live:

https://bedroesb.github.io/infectious-diseases-toolkit/

@bedroesb
bedroesb requested a review from rabuono as a code owner July 24, 2023 14:30
@bedroesb
bedroesb requested a review from a team as a code owner July 25, 2023 10:03
@bedroesb

bedroesb commented Jul 25, 2023

Copy link
Copy Markdown
Member Author

This is actually ready!

There is only a little but:
I added all tools from the spreadsheet and gave them IDs to make the transition progress of the upcoming pages easier. This means that in the all tools and resources list there will be tools that are not yet tagged in a page, because that page is on the way. Example: #141

@rabuono rabuono left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Great!

Approving here so there is no block, but I suggested some minor changes to the "Add new tool or resource" page.

Comment thread contribute/tool-resource-update.md Outdated
Comment thread contribute/tool-resource-update.md Outdated
Comment thread contribute/tool-resource-update.md Outdated
Comment thread contribute/tool-resource-update.md Outdated
Comment thread contribute/tool-resource-update.md Outdated
Comment thread contribute/tool-resource-update.md Outdated
Comment thread contribute/tool-resource-update.md Outdated
Comment thread contribute/tool-resource-update.md Outdated
Comment thread contribute/tool-resource-update.md Outdated
Comment thread contribute/tool-resource-update.md Outdated
bedroesb and others added 8 commits August 24, 2023 13:27
Co-authored-by: rafael buono <77321541+rabuono@users.noreply.github.com>
Co-authored-by: rafael buono <77321541+rabuono@users.noreply.github.com>
Co-authored-by: rafael buono <77321541+rabuono@users.noreply.github.com>
Co-authored-by: rafael buono <77321541+rabuono@users.noreply.github.com>
Co-authored-by: rafael buono <77321541+rabuono@users.noreply.github.com>
Co-authored-by: rafael buono <77321541+rabuono@users.noreply.github.com>
Co-authored-by: rafael buono <77321541+rabuono@users.noreply.github.com>
Co-authored-by: rafael buono <77321541+rabuono@users.noreply.github.com>
@bedroesb

Copy link
Copy Markdown
Member Author

Thanks a lot!

@bedroesb
bedroesb merged commit 6bbbff9 into elixir-europe:main Aug 24, 2023
@bedroesb

Copy link
Copy Markdown
Member Author

here we go! No random errors anymore!

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.

Move to new way of tagging tools

2 participants