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

Add setup requirements to training materials page #761

Merged
merged 2 commits into from
Aug 30, 2024

Conversation

paigem
Copy link
Contributor

@paigem paigem commented Aug 29, 2024

Description

Add Training Day setup requirements in markdown format

Closes #755

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue, e.g. dead links)
  • New link / content

Checklist:

  • The new content is accessible and located in the appropriate section
  • My changes do not break navigation and do not generate new warnings
  • I have checked that the links are valid and point to the intended content
  • I have checked my code/text and corrected any misspellings
  • I have chosen the correct tag for the level of support provided

When your pull request is ready please request a review.

Unless there is a specific person you want your PR to be reviewd by, please select the WebOps team: ACCESS-NRI/WebOps. This ensures the load for reviewing pull requests is shared equitably.

Copy link

github-actions bot commented Aug 29, 2024

PR preview
⚠️ There was an error in the pr-preview deployment. For more information, please check the Actions tab.
2024-08-30 11:00 AEST

@paigem paigem requested a review from a team August 29, 2024 23:32
Copy link
Contributor

@flicj191 flicj191 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 :)

@atteggiani
Copy link
Contributor

atteggiani commented Aug 30, 2024

Thank you @paigem!

Added a commit with minor changes:

  • Added links to projects
  • Minor formatting changes

I did it myself for quickness. Please check if it makes sense and you are fine with it, otherwise you're more than welcome to just remove my commit! :)

Other than that, looks pretty good!

@KAUR1984
Copy link
Contributor

Looks Awesome! Thanks for your work on this @paigem!

@paigem
Copy link
Contributor Author

paigem commented Aug 30, 2024

Looks great, thanks @atteggiani! 😄

@atteggiani
Copy link
Contributor

I'll rebase and merge.

@atteggiani atteggiani merged commit 2747658 into development Aug 30, 2024
1 of 3 checks passed
@atteggiani atteggiani deleted the paigem-patch-1 branch August 30, 2024 01:00
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.

TRAINING 2024: Include the setup requirements in the training tab
4 participants