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

Sfp 104 nl #52

Merged
merged 43 commits into from
Nov 27, 2023
Merged

Sfp 104 nl #52

merged 43 commits into from
Nov 27, 2023

Conversation

ToonVanDaele
Copy link
Collaborator

@ToonVanDaele ToonVanDaele commented May 10, 2022

Description

Add existing protocol sfp-104-nl. Imported from existing docx

Related Issue

Task list

Steps by contributor:

  • Add description to this pull request (under "## Description")
  • Submit the pull request by clicking 'create pull request'
  • Mark the pull request as draft
  • Check whether the version number in the index.Rmd yaml section is of format yyyy.nn.dev
  • In case of a protocol created from a pre-existing docx protocol, check if all sections comply with current template for a new protocol
  • Add further commits if needed and push them to GitHub
  • Update the protocol-specific NEWS.Rmd
  • Mark the pull request as 'ready for review'

Review steps for the author(s):

  • Add reviewers, at least one subject-matter specialist and one administrator
  • Wait for review comments and address them
  • Iterate until reviewer approvals (merging the pull request will be done by an administrator)

To be done by an administrator after review: see guidelines for admins.

Copy link
Collaborator

@hansvancalster hansvancalster left a comment

Choose a reason for hiding this comment

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

Er zijn een aantal bestanden die verwijderd moeten worden:

  • 08_afhankelijkheden_(indien van toepassing).Rmd
  • 10_bijlage_1
  • 12_bijlage_2

Kan je ook 12_appendices.Rmd controleren? De inhoud van de bijlagen ontbreekt denk ik.

@hansvancalster
Copy link
Collaborator

Om van de referenties in 11_referenties.Rmd een automatisch gegenereerde referentielijst te maken, kan je werken met markdown syntax om naar documenten te refereren die in een bibliografiebestand staan (zie https://inbo.github.io/tutorials/tutorials/r_citations_markdown/).
Hier is wat er daarvoor moet gebeuren:

  • een bestand referenties.yaml of referenties.bib toevoegen (afhankelijk van exportmogelijkheden van je referentiemanager - indien zotero, graag .yaml kiezen). Indien je met zotero werkt, mag dit initieel een leeg bestand zijn waarna je via de visual mode in RStudio connectie maakt met zotero en de betreffende documenten invoert (laat iets weten als dit onduidelijk is, dan leg ik dat wel eens live uit - zie ook tutorial on citation in R Markdown needs clarification tutorials#311)
  • in index.Rmd in de yaml sectie bibliography: referenties.yaml toevoegen
  • overige Rmd bestanden checken op verwijzingen en deze vervangen door markdown syntax @bibkey
  • in 11_referenties.Rmd alles verwijderen behalve de titel en een lijn toevoegen met <div id="refs"></div>

Dit zal ook voor je andere PRs nodig zijn.

@hansvancalster hansvancalster mentioned this pull request May 18, 2022
19 tasks
@hansvancalster
Copy link
Collaborator

@ToonVanDaele kan je de laatste wijzigingen eens binnenhalen en je protocol renderen: protocolhelper::render_protocol("sfp-104-nl"). Ga er dan nog eens grondig door om de laatste punten en komma's aan te pakken. De meeste kruisverwijzingen heb ik al opgelost, maar voor de verwijzingen naar paragrafen was het mij niet direct duidelijk welke paragraaf juist bedoeld wordt aangezien de nummering veranderd is. Dus zoek zeker op '§'. Zie https://bookdown.org/yihui/bookdown/cross-references.html voor uitleg hoe je hiernaar moet verwijzen. Daarna is het klaar wat mij betreft.

@ToonVanDaele
Copy link
Collaborator Author

  • § paragrafen nakijken
  • depencies toevoegen met add_depencies (komt in de yaml) (eerst alledrie publiceren)

@ToonVanDaele ToonVanDaele marked this pull request as ready for review March 9, 2023 11:22
@hansvancalster hansvancalster self-requested a review November 27, 2023 18:34
@hansvancalster hansvancalster self-requested a review November 27, 2023 18:44
@hansvancalster hansvancalster merged commit 599e4dc into main Nov 27, 2023
3 of 4 checks passed
@hansvancalster hansvancalster deleted the sfp-104-nl branch November 27, 2023 18:52
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.

3 participants