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

chore: Several small cleanup amends around K named components #3449

Merged
merged 2 commits into from
Jan 22, 2025

Conversation

johncowen
Copy link
Contributor

I went through and moved (almost) any remaining K components to use X components instead.

We still use K components in two instances:

  1. In Onboarding (we are removing onboarding soon remove onboarding #3386 so I didn't bother doing this here)
  2. KTable usage in a view in the Policy module. There are ongoing ideas for how we go about possibly doing something with KTable and AppCollection which may or may not end up being XTable and <XCollection :variant="table">

I noticed a couple of other things whilst doing this and amended here seeing as this is one of those "random bits" PRs, please ask if anything isn't clear 👍

@johncowen johncowen requested a review from a team as a code owner January 22, 2025 15:11
@johncowen johncowen requested review from schogges and removed request for a team January 22, 2025 15:11
Copy link

netlify bot commented Jan 22, 2025

Deploy Preview for kuma-gui ready!

Name Link
🔨 Latest commit 62840f2
🔍 Latest deploy log https://app.netlify.com/sites/kuma-gui/deploys/67912a06b527e30008e47b0b
😎 Deploy Preview https://deploy-preview-3449--kuma-gui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@schogges schogges left a comment

Choose a reason for hiding this comment

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

LGTM, left a Q

packages/kuma-gui/src/app/x/components/x-icon/XIcon.vue Outdated Show resolved Hide resolved
@johncowen johncowen force-pushed the chore/k-cleanup branch 2 times, most recently from 37a8efa to 03f2378 Compare January 22, 2025 17:09
Copy link
Contributor

@schogges schogges left a comment

Choose a reason for hiding this comment

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

🚀

@johncowen johncowen merged commit 2757cb6 into kumahq:master Jan 22, 2025
17 checks passed
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.

2 participants