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
fix(argo-cd): Pre-set statusbadge.url
only when statusbadge is enabled
#2658
Conversation
78a6d5b
to
645c5ba
Compare
This is just a small tweak of #2656: pre-populate |
Do you face issues when it is set but disabled? Or is it just cosmetics from your opinion? Xref: #2657 |
No issues, just cosmetics. In the previous iteration, one could consider it misleading or confusing if the badge url was pre populated, since badges are disabled by default. |
8102989
to
47e0c59
Compare
- Add `statusbadge.enabled` to values.yaml as `false` - Pre-set `statusbadge.url` to `global.domain` URL only when `statusbadge.enabled` is `true` - Update README.md - Fix a minor comment typo Signed-off-by: Bojan Raic <code@bojan.io>
47e0c59
to
a139834
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.
👍
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.
Thank you for your contribution! LGTM.
statusbadge.url
toglobal.domain
URL only whenstatusbadge.enabled
istrue
Checklist: