Skip to content
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

Unify env loading for apps #313

Open
7 tasks
ygrishajev opened this issue Aug 16, 2024 · 2 comments
Open
7 tasks

Unify env loading for apps #313

ygrishajev opened this issue Aug 16, 2024 · 2 comments
Assignees
Labels
tech-debt Technical debt

Comments

@ygrishajev
Copy link
Contributor

This commit introduces a certain env loading logic into an application which is described in apps/api/README.md#environment-variables.

We need to spread the same approach to other applications.

  • Create a new package env-loader which should basically expose the logic in this file as a function or a class method
  • Update api to use the package

Update every app to use the package and create all the required env files with proper values, update .dockerignore and gitignore, make sure sample file is filled with all used vars. Use api commit as reference.

  • deploy-web
  • indexer
  • provider-console
  • provider-proxy
  • stats-web

Required env files:
apps/api/env/.env
apps/api/env/.env.production
apps/api/env/.env.staging
apps/api/env/.env.mainnet
apps/api/env/.env.sandbox
apps/api/env/.env.sample

@ygrishajev ygrishajev added the tech-debt Technical debt label Aug 16, 2024
@baktun14
Copy link
Contributor

I think this might be a duplicate of this one #163

@ygrishajev
Copy link
Contributor Author

They overlap quite a bit but contain some differences. I'd leave this task as it's very specific and update the other one

@ygrishajev ygrishajev self-assigned this Sep 2, 2024
ygrishajev added a commit that referenced this issue Oct 3, 2024
Also add validated env configs to get essential network store vars

refs #163, #313
ygrishajev added a commit that referenced this issue Oct 3, 2024
ygrishajev added a commit that referenced this issue Oct 3, 2024
Also add validated env configs to get essential network store vars

refs #163, #313
ygrishajev added a commit that referenced this issue Oct 3, 2024
ygrishajev added a commit that referenced this issue Oct 8, 2024
Also add validated env configs to get essential network store vars

refs #163, #313
ygrishajev added a commit that referenced this issue Oct 8, 2024
ygrishajev added a commit that referenced this issue Oct 8, 2024
Also add validated env configs to get essential network store vars

refs #163, #313
ygrishajev added a commit that referenced this issue Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech-debt Technical debt
Projects
None yet
Development

No branches or pull requests

2 participants