Skip to content
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

Add reconciled and child object API endpoints and KustomizationDetail page #1489

Merged
merged 11 commits into from Feb 24, 2022

Conversation

jpellizzari
Copy link
Contributor

@jpellizzari jpellizzari commented Feb 18, 2022

Closes #1453

Adds the endpoint necessary to render the KustomizationDetail page, and the page itself to the UI:
Screenshot from 2022-02-22 14-18-42

These endpoints are mostly copy-paste from the pkg/server/server.go endpoints with some basic test coverage added.

@jpellizzari jpellizzari force-pushed the 1453-graph-endpoints branch 2 times, most recently from 819f30e to 08140ad Compare February 22, 2022 22:16
@jpellizzari jpellizzari changed the title 1453 graph endpoints Add reconciled and child object API endpoints and KustomizationDetail page Feb 22, 2022
@jpellizzari jpellizzari force-pushed the 1453-graph-endpoints branch 2 times, most recently from 958f98d to 32a1f0c Compare February 22, 2022 22:22
@jpellizzari jpellizzari assigned yitsushi and unassigned yitsushi Feb 22, 2022
@jpellizzari jpellizzari marked this pull request as ready for review February 22, 2022 22:42
@jpellizzari jpellizzari removed the request for review from JamWils February 22, 2022 22:55
ui/pages/v2/KustomizationDetail.tsx Show resolved Hide resolved
ui/pages/v2/KustomizationDetail.tsx Show resolved Hide resolved
ui/pages/v2/KustomizationDetail.tsx Show resolved Hide resolved
ui/pages/v2/KustomizationDetail.tsx Show resolved Hide resolved
@jpellizzari jpellizzari force-pushed the 1453-graph-endpoints branch 7 times, most recently from d2aa20b to ec844df Compare February 24, 2022 17:16
@jpellizzari
Copy link
Contributor Author

I tried to find the right go.mod incantation to get this to pass snyk, but gave up. The vulns are only medium.

@jpellizzari jpellizzari merged commit 2617b8c into v2 Feb 24, 2022
@jpellizzari jpellizzari deleted the 1453-graph-endpoints branch February 24, 2022 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants