Skip to content

add files to tutorial - #382

Open
sverbesselt01 wants to merge 3 commits into
inbo:masterfrom
sverbesselt01:Introduction_to_openeo_package
Open

sverbesselt01 wants to merge 3 commits into
inbo:masterfrom
sverbesselt01:Introduction_to_openeo_package

Conversation

@sverbesselt01

Copy link
Copy Markdown

add files to tutorial

Description

Related Issue

Task list

  • My tutorial or article is placed in a subfolder of tutorials/content/
  • The novel tutorial has a meaningful name, in relation to the content of the tutorial.
  • The filename of my tutorial or article is index.md. In case of an Rmarkdown tutorial I have knitted my index.Rmd to index.md (both files are pushed to the repo).
  • yaml header:
    • (recommended) I am included as author in the authors yaml tag, using [MY_AUTHOR_ID]. An author information file exists in <tutorials>/data/authors/<author>.toml.
    • I have added categories to the YAML header and my category tags are from the list of categories.
    • I have included meaningful and applicable tags (i.e. keywords) in the YAML header to improve the visibility of the new tutorial (see the tags listed in the tutorials website side bar).
    • The date is in format YYYY-MM-DD and adjusted.
  • (recommended) I have previewed this PR locally (see steps below; ask previous contributors for help) and confirmed that the new content renders as expected.

Previewing the pull request

Thanks to GitHub Actions, an artifact (=zip file) of the rendered website is automatically created for each pull request.
This provides a way to preview how these updates will look on the website, useful to contributors and reviewers.

Instructions to preview the updated website

  1. On the PR page, you can find a "details" link under "checks - On PR, build the site and ...". Go there, click on the top link in the left sidebar ("Summary"), and download the generated artifact at the bottom of the page.
  2. Decompress it into a target directory, e.g. Downloads/tutorials_preview.
  3. To preview the website, use a program which can serve http sites on your local machine. One such option is the servr package in R: & '\C:\Program Files\R\R-4.4.2\bin\Rscript.exe' -e "servr::httd('./tutorials_preview')" -p8887 (make sure to adjust the path to your Rscript.exe; on Linux, simply use Rscript -e [...]).
  4. Point your browser to http://localhost:8887.
  5. Review the updated website. As a contributor, you can push extra commits to update the PR. As a reviewer, you can accept/refuse/comment the PR.

Note: for step 3, you can use any other simple HTTP server to serve the current directory, e.g. Python http.server: python -m http.server 8887 --bind localhost --directory path/to/tutorials_preview

Alternative: Locally Building the Site

Alternatively, you can build the entire site locally (see the README for instructions); the Hugo preview server will update changes on the fly.
This requires Hugo to be installed on your computer.

add files to tutorial
Comment thread content/tutorials/introduction_to_openeo_package/index.qmd
Comment thread content/tutorials/introduction_to_openeo_package/index.qmd
Updated author name to lowercase and added error notes for authentication issues.

@bserrien bserrien left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you for this tutorial Sebastiaan! I verified that the website builds correctly so you can merge it with the main branch.

@sverbesselt01

Copy link
Copy Markdown
Author

Thank you for this tutorial Sebastiaan! I verified that the website builds correctly so you can merge it with the main branch.

Can I merge the branch (I don't see this option on github)? Or will you merge them?

@damianooldoni

Copy link
Copy Markdown
Member

@bserrien : I think @sverbesselt01 has no rights to merge a PR to master branch. Do you?

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