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
Add beacon build args #179
Conversation
This pull request has been linked to Shortcut Story #13580: Add REACT_APP variables to workflow. |
REACT_APP_GIT_REVISION=${{ steps.vars.outputs.revision }} | ||
REACT_APP_SENTRY_DSN=${{ secrets.REACT_APP_SENTRY_DSN }} | ||
REACT_APP_SENTRY_ENVIRONMENT=production | ||
REACT_APP_USE_DASH_LOCALE=false |
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.
It seems that the previous PR didn't actually add the QUARTERDECK_BASE_URL
and TENANT_BASE_URL
variables, not sure why but hopefully this works?
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.
@masskoder are you taking advantage of these environment variables in the Beacon UI code base?
REACT_APP_GIT_REVISION=${{ steps.vars.outputs.revision }} | ||
REACT_APP_SENTRY_DSN=${{ secrets.REACT_APP_SENTRY_DSN }} | ||
REACT_APP_SENTRY_ENVIRONMENT=production | ||
REACT_APP_USE_DASH_LOCALE=false |
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.
@masskoder are you taking advantage of these environment variables in the Beacon UI code base?
@pdeziel I removed my suggestions - it looks like we already have |
Scope of changes
This adds the rest of the beacon UI build args so that the variables are accessible in the production container.
Fixes SC-13580
Type of change
Acceptance criteria
All required variables are present and the github secrets have also been added.
Author checklist
Reviewer(s) checklist