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

ui: Hides the Routing tab for a service proxy #7195

Merged
merged 1 commit into from
Feb 3, 2020

Conversation

kaxcode
Copy link
Contributor

@kaxcode kaxcode commented Jan 31, 2020

  • Adds conditional in route to not make discovery-chain request if service kind is equal to connect-proxy or mesh-gateway

  • Adds conditional in template to not show Routing tab if chain returns as null

  • Creates a new acceptance test to test the Routing tab not being displayed for a service proxy

  • Adds tabs to the services/show page object

@kaxcode kaxcode added the theme/ui Anything related to the UI label Jan 31, 2020
@kaxcode kaxcode requested a review from a team January 31, 2020 21:35
Copy link
Contributor

@johncowen johncowen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @kaxcode !

I gave this a quick go and it looks great, awesome work! 👏

I left one little comment, which you might want to quickly tweak before merging.

Other than that :shipit: !

ui-v2/tests/acceptance/dc/services/show-routing.feature Outdated Show resolved Hide resolved
* Adds conditional in route to not make discovery-chain request if service kind is equal to `connect-proxy` or `mesh-gateway`

* Adds conditional in template to not show Routing tab if `chain` returns as null

* Creates a new acceptance test to test the Routing tab not being displayed for a service proxy

* Adds `tabs` to the services/show page object
@kaxcode kaxcode force-pushed the feature/ui-hide-routing-tab-for-service-proxy branch from b358e7f to dda1cc0 Compare February 3, 2020 14:45
@kaxcode kaxcode merged commit c706089 into ui-staging Feb 3, 2020
@kaxcode kaxcode deleted the feature/ui-hide-routing-tab-for-service-proxy branch February 3, 2020 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
theme/ui Anything related to the UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants