rancher / kubewarden-ui

Kubewarden's User Interface
Apache License 2.0
11 stars 13 forks source link

Remove readme step from policy creation - add slide-in for readme #776

Closed jordojordo closed 4 days ago

jordojordo commented 1 week ago

Fix #775

This will remove the policy readme step and replaces it with a slide out component showing the readme.

I also updated the ordering of the configuration tabs, and removed the module input if the policy is not custom.

https://github.com/rancher/kubewarden-ui/assets/40806497/c705361c-a450-440d-8fc6-eadbb1ff50e9

kravciak commented 6 days ago

PR works as expected, but I think removing module input is a problem.

With this change I can't see current policy version and I can't update policy from UI.

Also Show More button would be useful on policy edit page in addition to create page.

jordojordo commented 5 days ago

@kravciak

I think removing module input is a problem. With this change I can't see current policy version and I can't update policy from UI.

You can still see and update the module from the yaml view.

Also Show More button would be useful on policy edit page in addition to create page.

Good point, I added the Show More button to the policy edit page. Also I updated the wording to "Show Readme".