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

Add ability to disable colored logs; rename logging-related env vars #2068

Merged
merged 3 commits into from
Dec 3, 2024

Conversation

alco
Copy link
Member

@alco alco commented Nov 28, 2024

Related documentation update - #2067.

@alco alco changed the base branch from alco/log-colors-config to main November 29, 2024 09:54
@alco alco force-pushed the alco/env-var-renaming branch from caee2d7 to fedbbd4 Compare November 29, 2024 10:00
@alco alco changed the title WIP Rename env vars to avoid confusion between log output and shape logs Add ability to disable colored logs; rename logging-related env vars Nov 29, 2024
Copy link

netlify bot commented Nov 29, 2024

Deploy Preview for electric-next ready!

Name Link
🔨 Latest commit 94bd1a0
🔍 Latest deploy log https://app.netlify.com/sites/electric-next/deploys/674db1ce98f6e800088e5119
😎 Deploy Preview https://deploy-preview-2068--electric-next.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 your Netlify site configuration.

@balegas
Copy link
Contributor

balegas commented Dec 2, 2024

Hei, please modify the SST config to use this option

@alco alco force-pushed the alco/env-var-renaming branch from fedbbd4 to 94bd1a0 Compare December 2, 2024 13:10
@alco
Copy link
Member Author

alco commented Dec 2, 2024

Hei, please modify the SST config to use this option

Electric's config is not used when it is started in multi-tenancy mode. So no changes are needed to cloud's SST config. Unless you were referring to the two sst.config.ts files we have in two example apps in this repo?

@icehaunter icehaunter merged commit 5b1c3e6 into main Dec 3, 2024
26 checks passed
@icehaunter icehaunter deleted the alco/env-var-renaming branch December 3, 2024 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants