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

Discussion: Instead of using text_snippet wip for sections individually, should we just use add it to _quarto.yml? #25

Closed
signekb opened this issue Jan 21, 2025 · 1 comment · Fixed by #26
Assignees

Comments

@signekb
Copy link
Member

signekb commented Jan 21, 2025

Could also apply to a more general discussion: Should all courses/tutorials that are wip just have a wip callout in the _quarto.yml like we do for Sprout?

body-header: |
    ::: {.callout-warning appearance="default" icon="true"}
    🚧 Sprout is still in active development and evolving quickly, so the
    documentation and functionality may not work as described
    and could undergo substantial changes 🚧
    :::

https://github.com/seedcase-project/seedcase-sprout/blob/7fb90325654c00c929eb13c8b011d997f027fa99/_quarto.yml#L18L23

@signekb signekb moved this from Todo to In review in Developing new courses Jan 21, 2025
@signekb signekb self-assigned this Jan 21, 2025
@lwjohnst86
Copy link
Member

Yea, that works! Can't use the text snippets inside the quarto yml I think though, maybe try it?

@github-project-automation github-project-automation bot moved this from In review to Done in Developing new courses Jan 22, 2025
lwjohnst86 pushed a commit that referenced this issue Jan 22, 2025
## Description

This PR adds the wip `text_snippet` to `_quarto.yml` and removes it from
the individual docs.

<img width="698" alt="image"
src="https://github.com/user-attachments/assets/ffd5cf1d-992b-425f-ba5d-222a76c39470"
/>


Closes #25 

<!-- Please delete as appropriate: -->
This PR needs a quick review.

## Checklist

- [X] Ran `just run-all`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants