-
Notifications
You must be signed in to change notification settings - Fork 23
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
feat: policies list in the quick start page. #424
Conversation
Adds an embedded HTML with the policies available in the ArtifactHub Signed-off-by: José Guilherme Vanz <[email protected]>
✅ Deploy Preview for silly-bunny-8cedd0 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Looks nice, I left a minor fix
docs/quick-start.md
Outdated
chapters](tutorials/writing-policies/rego/01-intro-rego.md). Here is the list of | ||
available policies on ArtifactHub: |
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.
chapters](tutorials/writing-policies/rego/01-intro-rego.md). Here is the list of | |
available policies on ArtifactHub: | |
chapters](tutorials/writing-policies/rego/01-intro-rego.md). | |
Finally, here is the list of available policies on ArtifactHub. |
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 think the @jhkrug approach a better way to list the policies and makes this suggestion obsolete. Therefore, I'll skip this change. Ok? ;)
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 modulo Flavio's suggestion.
Co-authored-by: John Krug <[email protected]> Signed-off-by: José Guilherme Vanz <[email protected]>
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
I prefer @jhkrug's approach. You can go ahead and merge this PR |
@jvanz I think it would be cool to embed as a widget rather than linking to artifactHub. The idea is to make it easier for users to directly interact with Kubewarden policies directly on kubewarden.io. You can take a look at some examples here: |
Description
Adds an embedded HTML with the policies available in the ArtifactHub
Fix #401