Skip to content

MXWAR-36: Allow backend URL override via environment variable#30

Open
teena14 wants to merge 2 commits intoopenMF:devfrom
teena14:MXWAR-36-env-backend-config
Open

MXWAR-36: Allow backend URL override via environment variable#30
teena14 wants to merge 2 commits intoopenMF:devfrom
teena14:MXWAR-36-env-backend-config

Conversation

@teena14
Copy link

@teena14 teena14 commented Jan 23, 2026

Jira

MXWAR-36

Context

This PR adds an optional environment-variable-based override for the backend API URL while keeping the existing localhost fallback unchanged.

The intent is to improve developer convenience for frontend and UI/UX contributors who may not want to set up a full local Fineract backend, without changing the default or recommended workflow.

Changes

  • Added support for overriding the backend base URL via environment variables
  • Preserved existing localhost fallback behavior
  • No runtime behavior changes by default

Notes

  • This change is fully backward-compatible
  • Default behavior remains unchanged unless an env variable is explicitly provided

Copy link
Collaborator

@alberto-art3ch alberto-art3ch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please see my comment

vite.config.ts Outdated
proxy: {
"/api": {
target:
env.VITE_FINERACT_API_URL ||
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we use more standard env variable name, I mean without VITE_

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes we can actually name it without VITE_ , i will do so and update it

@DavidH-1
Copy link
Contributor

@alberto-art3ch are you happy with the changes. @teena14 made?

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.

3 participants