Skip to content

add view for showing the associated gateways #3

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

Open
wants to merge 3 commits into
base: gh-245
Choose a base branch
from

Conversation

Anton-Fil
Copy link
Contributor

closes#244

@guicassolato guicassolato moved this to Todo in Kuadrant Aug 1, 2025
@Anton-Fil Anton-Fil requested a review from R-Lawton August 1, 2025 09:43
@R-Lawton
Copy link

R-Lawton commented Aug 7, 2025

Hey @Anton-Fil,
Im seeing a few issues with the gateway view. The first is that the view is trying to show gateways that don't exist, and secondly, it's showing duplicate gateways.

I'm also thinking we don't need two separate functions one for gateways one for httproutes. My original idea of the one combined i think would be cleaner to read and less resource heavy. If you would like me to run through it with you let me know, im happy to pair program on it with you.

Screenshot 2025-08-07 at 14 46 52 Screenshot 2025-08-07 at 14 46 16

@R-Lawton R-Lawton mentioned this pull request Aug 7, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

2 participants