-
Notifications
You must be signed in to change notification settings - Fork 44
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: add operation to list all clusters #717
feat: add operation to list all clusters #717
Conversation
Your org requires the Graphite merge queue for merging into mainYou must have a Graphite account in order to use the merge queue. Sign up using this link. You can enable merging using labels in your Graphite merge queue settings. |
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @AngelOnFira and the rest of your teammates on |
27f1de1
to
26911c8
Compare
d4ec784
to
1152d87
Compare
26911c8
to
081ed24
Compare
1152d87
to
e07ddb3
Compare
ed5d04e
to
f2343ed
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.
What is this operation going to be used for?
It seems like the way to go is to do as much on the api consumer (bolt in this case) as possible. As such, there needs to be a way to convert from the I was chatting with @NathanFlurry about this, and the idea is putting as much of the logic on the client side as possible. But if this doesn't seem right, I can check out a different way of going about it. |
f2343ed
to
83192e4
Compare
83192e4
to
4bdfefc
Compare
Merge activity
|
<!-- Please make sure there is an issue that this PR is correlated to. --> ## Changes <!-- If there are frontend changes, please include screenshots. -->
4bdfefc
to
1f4b169
Compare
Changes