-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update Content development guide #114
Conversation
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.
I really appreciate you updated the content in almost all places (e.g. removing mention of the Argo artifacts issue which is already resolved) 👍 Apart from some minor suggestions, the biggest things I'm missing are:
- a description of an Argo artifact content, that is: which properties it should have -
value
,backend
fields - to be able to upload it as a TypeInstance. I'm not seeing it anywhere 🤔 - a short, dedicated section how to use the storage backends from content developer perspective. I mean - really short and concise, so it shouldn't be a big deal 🙂
9744eb7
to
b975ddd
Compare
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.
Please make sure that all manifests are up to date according to our main
branch on hub-manifests (including the
unpackValue: true` from jinja) and apply all changes. Then it will be ready to be merged 👍
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.
Please apply last two comments and we can merge it 🚀
Description
Changes proposed in this pull request:
Generally, I went through the whole text/examples and update it.
Related issue(s)