-
Notifications
You must be signed in to change notification settings - Fork 21
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
feat(zones): move KDSSubscriptions to use a table/summary #2664
Conversation
✅ Deploy Preview for kuma-gui ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
e434257
to
2ce1472
Compare
97d05a1
to
ad69dd6
Compare
I figured we probably don't want to show the YAML, its not like you'd want to use it for anything. If that incorrect we can always add as a follow up. |
f3c2558
to
c895162
Compare
c895162
to
e20326a
Compare
c0069cc
to
7874509
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some nits but I like the new view it looks more powerful
5eafdc3
to
66ef964
Compare
K this one has had a few alterations after a first round of reviews, good to go again now 👍 |
src/app/subscriptions/views/SubscriptionSummaryOverviewView.vue
Outdated
Show resolved
Hide resolved
76ac0a2
to
06e4fca
Compare
Signed-off-by: John Cowen <[email protected]>
Signed-off-by: John Cowen <[email protected]>
Signed-off-by: John Cowen <[email protected]>
Signed-off-by: John Cowen <[email protected]>
Signed-off-by: John Cowen <[email protected]>
06e4fca
to
77aaf2b
Compare
LGTM do we have issues to cover the same work for dataplane and zone proxies? If not can we open them and milestone them for 2.10 please? |
Changes Zone Subscriptions only to use a table/summary view/interaction.
We decided that there would be far more space to show things if we used our common table/summary panel pattern instead of the accordion where there is very little space to show things:
See:
Questions:1. Do we want to show the Subscription YAML, the Subscription JSON config, or both?2. If we want to show both, should we add a separate[YAML]
tab or use theStructured/YAML
dropdown?Note:
I'm guessing we'll want the same layout for DPP subscriptions, I plan to do this as a follow up and there will some changes made on top of here to make the panel work in both places.
Closes #2704