Kuadrant / kuadrant-console-plugin

Apache License 2.0
2 stars 2 forks source link

Add policies page #35

Closed david-martin closed 1 month ago

david-martin commented 1 month ago

Closes #17

TODO:

david-martin commented 1 month ago

I'd like to checkpoint the work in this PR and get it merged if possible as I believe it satisfies the majority of the ask and aligns to the mockups for the most part. Screenshots of the view included below, and a list of some things that can be addressed in follow up PRs as more adjacent changes land (I could see us doing a larger combined review of changes and making a snag list for example).

All Policies page

image

Single Policy type page

image

Potential follow up changes

image image
david-martin commented 1 month ago

@jasonmadigan Rebased and ready for review.

jasonmadigan commented 1 month ago

👀

jasonmadigan commented 1 month ago

@david-martin ^

david-martin commented 1 month ago

@jasonmadigan Thanks for review. Ack on comments, with response on the CRD versioning one.

jasonmadigan commented 1 month ago

LGTM