Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Env variables cleanup#641

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
FalkWolsky merged 2 commits intodevfromenv-variables-cleanup
Jan 16, 2024
Merged

Env variables cleanup#641

FalkWolsky merged 2 commits intodevfromenv-variables-cleanup
Jan 16, 2024

Conversation

ludomikula
Copy link
Collaborator

Proposed changes

Docker image(s) and helm variables cleanup and rename. To make more sense in the environment variables, we renamed and unified them.As such this is a breaking change for your current docker-compose files!

Here is the table of all old variables names mapped to the new ones:

Old variable nameNew variable name
REDIS_ENABLEDLOWCODER_REDIS_ENABLED
REDIS_URLLOWCODER_REDIS_URL
MONGODB_ENABLEDLOWCODER_MONGODB_ENABLED
MONGODB_URLLOWCODER_MONGODB_URL
LOWCODER_MONGODB_EXPOSED
API_SERVICE_ENABLEDLOWCODER_API_SERVICE_ENABLED
LOWCODER_API_SERVICE_URLLOWCODER_API_SERVICE_URL
LOWCODER_API_KEY_SECRETLOWCODER_API_KEY_SECRET
LOWCODER_API_RATE_LIMIT
NODE_SERVICE_ENABLEDLOWCODER_NODE_SERVICE_ENABLED
LOWCODER_NODE_SERVICE_URLLOWCODER_NODE_SERVICE_URL
FRONTEND_ENABLEDLOWCODER_FRONTEND_ENABLED
PUIDLOWCODER_PUID
PGIDLOWCODER_PGID
DEFAULT_ORGS_PER_USERLOWCODER_MAX_ORGS_PER_USER
DEFAULT_ORG_MEMBER_COUNTLOWCODER_MAX_MEMBERS_PER_ORG
DEFAULT_ORG_GROUP_COUNTLOWCODER_MAX_GROUPS_PER_ORG
DEFAULT_ORG_APP_COUNTLOWCODER_MAX_APPS_PER_ORG
DEFAULT_DEVELOPER_COUNTLOWCODER_MAX_DEVELOPERS
LOWCODER_MAX_REQUEST_SIZELOWCODER_MAX_REQUEST_SIZE
LOWCODER_MAX_QUERY_TIMEOUTLOWCODER_MAX_QUERY_TIMEOUT
AUTH_EMAIL_ENABLELOWCODER_EMAIL_AUTH_ENABLED
ENABLE_USER_SIGN_UPLOWCODER_EMAIL_SIGNUP_ENABLED
COMMON_WORKSPACE_MODELOWCODER_WORKSPACE_MODE
LOWCODER_CREATE_SIGNUP_WORKSPACELOWCODER_CREATE_WORKSPACE_ON_SIGNUP
ENCRYPTION_PASSWORDLOWCODER_DB_ENCRYPTION_PASSWORD
ENCRYPTION_SALTLOWCODER_DB_ENCRYPTION_SALT
CORS_ALLOWED_DOMAINSLOWCODER_CORS_DOMAINS

Types of changes

What types of changes does your code introduce to Lowcoder?
Put anx in the boxes that apply.

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (if none of the other choices apply)

Checklist

You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help!
This is simply a reminder of what we are going to look for before merging your code.
Put anx in the boxes that apply.

  • Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • Any dependent changes have been merged and published in downstream modules

Further comments

This change breaks your existingdocker-compose and helmvalues.yaml because the environment variables were renamed!

@netlifyNetlify
Copy link

netlifybot commentedJan 16, 2024
edited
Loading

Deploy Preview forlowcoder-test ready!

NameLink
🔨 Latest commit27ca2d4
🔍 Latest deploy loghttps://app.netlify.com/sites/lowcoder-test/deploys/65a6b06539285800080ea2b9
😎 Deploy Previewhttps://deploy-preview-641--lowcoder-test.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to yourNetlify site configuration.

Copy link
Contributor

@FalkWolskyFalkWolsky left a comment

Choose a reason for hiding this comment

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

Impressive. What a wonderful contribution to develop step by step more clean structure of the project

@FalkWolskyFalkWolsky merged commit0d40d43 intodevJan 16, 2024
@FalkWolskyFalkWolsky deleted the env-variables-cleanup branchJanuary 16, 2024 18:06
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@FalkWolskyFalkWolskyFalkWolsky approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ludomikula@FalkWolsky

[8]ページ先頭

©2009-2025 Movatter.jp