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

Prepare docker-compose.yml for mail and calendar variables #10464

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

FatoYo
Copy link

@FatoYo FatoYo commented Feb 24, 2025

Adding the placeholders for the environment variables related to setting up the mail and calendar sync. This will make the Twenty setup easier for new users.

Adding the placeholders for the environment variables related to setting up the mail and calendar sync. This will make the Twenty setup easier for new users.
Copy link
Contributor

Welcome!

Hello there, congrats on your first PR! We're excited to have you contributing to this project.
By submitting your Pull Request, you acknowledge that you agree with the terms of our Contributor License Agreement.

Generated by 🚫 dangerJS against c2f270d

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

This PR enhances the Docker Compose configuration by adding comprehensive environment variables for mail and calendar integration support across both server and worker services.

  • Added Gmail and Microsoft provider toggles (MESSAGING_PROVIDER_GMAIL_ENABLED, CALENDAR_PROVIDER_GOOGLE_ENABLED, etc.) in /packages/twenty-docker/docker-compose.yml
  • Added email verification configuration (IS_EMAIL_VERIFICATION_REQUIRED, EMAIL_VERIFICATION_TOKEN_EXPIRES_IN) for user authentication
  • Added SMTP configuration variables (EMAIL_SMTP_HOST, EMAIL_SMTP_PORT, etc.) for email service setup
  • Added Microsoft OAuth variables (AUTH_MICROSOFT_CLIENT_ID, AUTH_MICROSOFT_CALLBACK_URL, etc.) for Microsoft integration

💡 (1/5) You can manually trigger the bot by mentioning @greptileai in a comment!

1 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile

Comment on lines +33 to +36
MESSAGING_PROVIDER_GMAIL_ENABLED: ${MESSAGING_PROVIDER_GMAIL_ENABLED}
CALENDAR_PROVIDER_GOOGLE_ENABLED: ${CALENDAR_PROVIDER_GOOGLE_ENABLED}
MESSAGING_PROVIDER_MICROSOFT_ENABLED: ${MESSAGING_PROVIDER_MICROSOFT_ENABLED}
CALENDAR_PROVIDER_MICROSOFT_ENABLED: ${CALENDAR_PROVIDER_MICROSOFT_ENABLED}
Copy link
Contributor

Choose a reason for hiding this comment

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

style: Consider grouping related environment variables (e.g., all Google-related vars together, all Microsoft-related vars together) for better maintainability

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.

1 participant