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

Added cal.com as template #2119

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

Conversation

kunumigab
Copy link
Contributor

Scheduling infrastructure for absolutely everyone.

Docker-Compose instructions used here: https://github.com/calcom/docker

Scheduling infrastructure for absolutely everyone.

Docker-Compose instructions used here: https://github.com/calcom/docker
@RayBB
Copy link
Contributor

RayBB commented May 2, 2024

@kunumigab
Not a maintainer but two small bits of feedback based on what I've seen in this repo:

  1. Usually the app service goes before the database service in the compose file (probably just so it's higher up when you edit variables)
  2. Usually template files end with a newline

Unfortunately, I can't test this right now since my main server is ARM.

@RayBB
Copy link
Contributor

RayBB commented May 13, 2024

@andrasbacsai any chance you could review this PR this week?
@kunumigab

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.

None yet

2 participants