-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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: load balance between environments #3500
Conversation
|
||
export const BLUEPRINTS_API_URL = | ||
window._env_.REACT_APP_BLUEPRINTS_API_URL || isCypress | ||
? window._env_.REACT_APP_BLUEPRINTS_API_URL || process.env.REACT_APP_BLUEPRINTS_API_URL || 'http://localhost:1336' |
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.
window._env_.REACT_APP_BLUEPRINTS_API_URL
allows to override that URL, should we keep that?
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.
we definitely can remove it, I left it is just in case we would want to override it.
}; | ||
|
||
export const POPULAR_TEMPLATES_GROUPED = | ||
process.env.NODE_ENV === 'production' ? production().templates : development().templates; |
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.
why do we want to differentiate?
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.
because the ids will be different in different environments.
…-blueprint-api # Conflicts: # apps/api/src/app/blueprint/usecases/get-grouped-blueprints/consts.ts
What change does this PR introduce?
make sure that all environments that are not prod will point to dev api.
Why was this change needed?
remove the load of the requests from prod api.
Other information (Screenshots)
the things that are still missing: