-
Notifications
You must be signed in to change notification settings - Fork 1.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
Replace all environment variables instead of only API_URL #5055
Conversation
🦋 Changeset detectedLatest commit: ff0c53c The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
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.
I think the improvement is great.
@krzysztofzuraw can you verify this against your latest work on dockerization?
nginx/replace-api-url.sh
Outdated
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.
Good job - works as it should - can you change the filename to replace-env-vars
instead of replace-api-url.sh
?
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.
of course
@yasinuslu thank you for your work 🙇🏻♂️ |
What type of PR is this?
Related Issues or Documents
Instead of replacing only the API_URL, docker image now replaces all environment variables available to the index.html
Usage Instructions, Screenshots, Recordings
Have you written tests?
replace-api-url.sh
it didn't have tests before, so I didn't add it. We can figure out a way to write a test for it if you request it.[Optional] Description