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

Fixed repo mgmt buttons showing in additional clusters #5175

Merged
merged 1 commit into from
Aug 9, 2022

Conversation

castelblanque
Copy link
Collaborator

@castelblanque castelblanque commented Aug 8, 2022

Signed-off-by: Rafa Castelblanque rcastelblanq@vmware.com

Description of the change

Trivial PR that fixes issue in the multicluster mode that was showing buttons for managing repos when on additional clusters. This actions are not valid in that scenario.

image

Those buttons are not appearing anymore in additional clusters.
image

Benefits

Users will not get the repository creation dialog on additional clusters -a feature not available in Kubeapps so far- and thus will not get an error when trying to create a repo.
image

Possible drawbacks

N/A

Signed-off-by: Rafa Castelblanque <rcastelblanq@vmware.com>
@netlify
Copy link

netlify bot commented Aug 8, 2022

Deploy Preview for kubeapps-dev canceled.

Name Link
🔨 Latest commit aedbfed
🔍 Latest deploy log https://app.netlify.com/sites/kubeapps-dev/deploys/62f0ed22cc78060008c30652

@@ -31,3 +31,6 @@ telepresence.log

# Use yarn for package management
package-lock.json

# React-app proxy
src/setupProxy.js
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Added the React app proxy to ignore.

Copy link
Contributor

@antgamdia antgamdia left a comment

Choose a reason for hiding this comment

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

Great, thanks!

@castelblanque castelblanque merged commit 8338ce6 into main Aug 9, 2022
@castelblanque castelblanque deleted the fix-multicluster-repo-buttons branch August 9, 2022 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants