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(issue-339): Adds config ability switch between branded content vs default #355

Merged
merged 10 commits into from
Apr 21, 2022

Conversation

pattishin
Copy link
Collaborator

@pattishin pattishin commented Apr 18, 2022

Depends on #356 to be merged first.
Resolves issue: #339

To verify locally:

  • Replace string in line 26 with your api container url (experimental/client-app/server/proxy.js)
npm i
npm run build
npm run dev

See that the header changes according to THEME environment variable hardcoded in package.json. Currently just 2 options: cymbal branding and default emblem branding.

@pattishin pattishin requested a review from a team as a code owner April 18, 2022 17:02
@pattishin pattishin added the tech: lit Related to the experimental lit frontend application label Apr 18, 2022
Base automatically changed from chore/issue-338 to main April 21, 2022 23:31
@pattishin pattishin merged commit 32cb675 into main Apr 21, 2022
@pattishin pattishin deleted the feat/issue-339-config-brand branch April 21, 2022 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech: lit Related to the experimental lit frontend application
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants