From 57218d14548bf1d4cd59f6976e84cf178023305d Mon Sep 17 00:00:00 2001 From: steffeydev Date: Mon, 5 Jun 2023 14:19:55 +0000 Subject: [PATCH] fix: allow icon changes configuration via env (#3567) `window.ALLOW_ICON_CHANGES` needs to be written as a boolean during the docker deploy instead of a string. The strings `"true"` and `"false"` both evaluate to `true` in JS, so we need to use the boolean `true` and `false` for the assertion in `Logo.vue` to be meaningful. Co-authored-by: SteffeyDev Reviewed-on: https://kolaente.dev/vikunja/frontend/pulls/3567 Reviewed-by: konrad Co-authored-by: steffeydev Co-committed-by: steffeydev --- docker/injector.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docker/injector.sh b/docker/injector.sh index 1958dbc7c..9aa1efadf 100755 --- a/docker/injector.sh +++ b/docker/injector.sh @@ -12,6 +12,6 @@ sed -ri "s:^(\s*window.API_URL\s*=)\s*.+:\1 '${VIKUNJA_API_URL}':g" /usr/share/n sed -ri "s:^(\s*window.SENTRY_ENABLED\s*=)\s*.+:\1 ${VIKUNJA_SENTRY_ENABLED}:g" /usr/share/nginx/html/index.html sed -ri "s:^(\s*window.SENTRY_DSN\s*=)\s*.+:\1 '${VIKUNJA_SENTRY_DSN}':g" /usr/share/nginx/html/index.html sed -ri "s:^(\s*window.PROJECT_INFINITE_NESTING_ENABLED\s*=)\s*.+:\1 '${VIKUNJA_PROJECT_INFINITE_NESTING_ENABLED}':g" /usr/share/nginx/html/index.html -sed -ri "s:^(\s*window.ALLOW_ICON_CHANGES\s*=)\s*.+:\1 '${VIKUNJA_ALLOW_ICON_CHANGES}':g" /usr/share/nginx/html/index.html +sed -ri "s:^(\s*window.ALLOW_ICON_CHANGES\s*=)\s*.+:\1 ${VIKUNJA_ALLOW_ICON_CHANGES}:g" /usr/share/nginx/html/index.html date -uIseconds | xargs echo 'info: started at'