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

feat(ui): control whether the Github logo is displayed #571

Merged
merged 2 commits into from
Aug 2, 2024

Conversation

z1cheng
Copy link
Contributor

@z1cheng z1cheng commented Aug 1, 2024

What type of PR is this?

/kind feature

What this PR does / why we need it:

Control whether the Github logo is displayed in UI

Which issue(s) this PR fixes:

Close #501

Signed-off-by: z1cheng <imchench@gmail.com>
Copy link

github-actions bot commented Aug 1, 2024

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@z1cheng
Copy link
Contributor Author

z1cheng commented Aug 1, 2024

I have read the CLA Document and I hereby sign the CLA

@elliotxx
Copy link
Collaborator

elliotxx commented Aug 2, 2024

@z1cheng Cool, let us review it. cc @hai-tian

@z1cheng
Copy link
Contributor Author

z1cheng commented Aug 2, 2024

@elliotxx Do we have any instructions/Makefile to run karpor locally for better testing and debugging the code? including creating a cluster(Kind?), building images, installing with helm?

@elliotxx
Copy link
Collaborator

elliotxx commented Aug 2, 2024

@z1cheng Of course, you can execute make help in the project root directory to view all targets, as well as view documents Code Contribute to view how to start locally. If you need, I will provide more details on local startup and debugging.

Copy link
Collaborator

@elliotxx elliotxx left a comment

Choose a reason for hiding this comment

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

lgtm

@elliotxx elliotxx marked this pull request as ready for review August 2, 2024 06:01
@elliotxx elliotxx added the area/experience Experience related optimization label Aug 2, 2024
@elliotxx elliotxx merged commit 6bd5fc5 into KusionStack:main Aug 2, 2024
7 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Aug 2, 2024
@z1cheng z1cheng deleted the github_badge branch August 2, 2024 08:19
@elliotxx elliotxx added this to the v0.5.0 milestone Aug 3, 2024
@elliotxx
Copy link
Collaborator

elliotxx commented Aug 4, 2024

@z1cheng Hi, I have released version 0.4.6 with this feature included.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/experience Experience related optimization
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Experience: Whether the GitHub Badge is displayed through the flag in /server-configs
2 participants