-
Notifications
You must be signed in to change notification settings - Fork 0
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
BC-8404 - Move tldraw config to issolate it from server config #42
base: main
Are you sure you want to change the base?
Conversation
Quality Gate passedIssues Measures |
@@ -6,16 +6,13 @@ metadata: | |||
labels: | |||
app: tldraw-server | |||
data: | |||
API_HOST: "http://api-svc:3030" | |||
WS_PATH_PREFIX: "/tldraw-server" | |||
WS_PORT: "3345" |
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.
What happend to the WS_PORT?
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.
Is default in code
@@ -6,16 +6,13 @@ metadata: | |||
labels: | |||
app: tldraw-server | |||
data: | |||
API_HOST: "http://api-svc:3030" |
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.
What happend to the API_HOST?
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.
rename to AUTHORIZATION_API_HOST
TLDRAW__ASSETS_ALLOWED_MIME_TYPES_LIST: "image/png,image/jpeg,image/gif,image/svg+xml" | ||
FEATURE_TLDRAW_ENABLED: "true" | ||
NEST_LOG_LEVEL: "info" | ||
EXIT_ON_ERROR: "true" |
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.
no more exit on error?
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.
Is default true in code
Description
Links to Tickets or other pull requests
https://ticketsystem.dbildungscloud.de/browse/BC-8404
BC-8404 - Move tldraw config to issolate it from server config tldraw-client#118
BC-8404 - Move tldraw config to issolate it from server config dof_app_deploy#1039
Changes
Datasecurity
Deployment
New Repos, NPM pakages or vendor scripts
Screenshots of UI changes
Approval for review