Skip to content

feat: removed hard-coded configuration parameters in preparation for automated client deployment - #65

Merged
bakirFS merged 19 commits into
mainfrom
feat/switch-to-github-variables
Aug 7, 2026
Merged

feat: removed hard-coded configuration parameters in preparation for automated client deployment#65
bakirFS merged 19 commits into
mainfrom
feat/switch-to-github-variables

Conversation

@bakirFS

@bakirFS bakirFS commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Contributes to #59

Created this PR to remove hard-coded variables from code which caused us to have bespoke deployments for each customer using private cloud.

You need to provide 4 values at all time in GitHub variables + FORGE_API_TOKEN in secrets.
Those are: FLAGSMITH_BACKEND_HOST, FORGE_APP_ID, FLAGSMITH_APP and FLAGSMITH_API_V1.

Changes have been tested by providing invalid manifest variables (FLAGSMITH_BACKEND_HOST) which made the deployment fail and by providing invalid runtime variables (FLAGSMITH_API_V1) which deployed the app but not communicate with the API.

@bakirFS
bakirFS requested a review from a team as a code owner July 17, 2026 14:18
@bakirFS
bakirFS requested review from talissoncosta and removed request for a team July 17, 2026 14:18
Prevents the encryption prompt from failing the deploy in CI's non-TTY
environment.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@bakirFS
bakirFS merged commit d2ce350 into main Aug 7, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants