-
Notifications
You must be signed in to change notification settings - Fork 41
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
RHIDP-1542: Document adding templates #219
Conversation
a15e1e2
to
a02ac31
Compare
20cc75d
to
3c6c6f1
Compare
a8cc8df
to
17b73d9
Compare
I have no issue with the content and seems accurate to me however, is the template editor our recommended approach to templates? Shouldn't it be storing them in GitHub/GitLab, etc and using the Catalog processors to ingest them to have a single source of truth? |
17b73d9
to
24a809c
Compare
Reworking per @jayfray12 's suggestion |
2924964
to
3a028e0
Compare
Why are you packaging the documentation which is interesting for the developers or platform engineers part of the admin guide ? |
It didn't seem to me like something that would fit within the installation or getting started guides since it requires modifying config files. Where would you prefer it to be included? Also, it would have been good to have included this feedback in #201 instead, because that's where the assembly was actually added. This PR is just expanding on that now existing section. |
Ideally that should be part of the guide used by the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One minor change, otherwise lgtm
3a028e0
to
ccfdb4e
Compare
ccfdb4e
to
469834b
Compare
/cherrypick 1.1.x |
/cherrypick 1.2.x |
@abrennan89: new pull request created: #314 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
@abrennan89: new pull request created: #315 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Versions:
1.1+
Jira link:
Preview link:
https://redhat-developer.github.io/red-hat-developers-documentation-rhdh/pr-219/admin-rhdh/#proc-adding-templates_assembly-admin-templates
Reviews: