-
Notifications
You must be signed in to change notification settings - Fork 7.1k
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
Support CYMBAL_BRANDING environment variable #602
Conversation
🚲 PR staged at http://35.231.157.100 |
🚲 PR staged at http://35.231.157.100 |
a8a05f2
to
fb36f5b
Compare
🚲 PR staged at http://35.231.157.100 |
1 similar comment
🚲 PR staged at http://35.231.157.100 |
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.
Looks good! Some minor thoughts left as comments.
Also have we discussed about running an instance of the repo with the Cymbal Branding
as well like the onlineboutique site?
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.
Tested on my cluster + working. All good, only request is to remove the changes to release/
.
fb36f5b
to
f89be01
Compare
🚲 PR staged at http://35.231.157.100 |
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.
LGTM!
All issues raised by reviewers are addressed. Merging. |
* Support CYMBAL_LOGO env. variable * Add documentation for CYMBAL_LOGO * Allow case variation in CYMBAL_LOGO value * Rename CYMBAL_LOGO to CYMBAL_BRANDING
See the "What needs to be done?" section of #574.
Background
CYMBAL_BRANDING
to true (in thefrontend
microservice), similar to Bank of Anthos.Change Summary
This pull-request
CYMBAL_BRANDING
environment variable.CYMBAL_BRANDING
indocs/cymbal-shops.md
.Additional Notes
CYMBAL_BRANDING
is set to"true"
).