Skip to content

feat(template): add Formbricks template with Docker Compose and environment setup#1237

Merged
Siumauricio merged 5 commits intoDokploy:canaryfrom
bernabedev:canary
Jan 31, 2025
Merged

feat(template): add Formbricks template with Docker Compose and environment setup#1237
Siumauricio merged 5 commits intoDokploy:canaryfrom
bernabedev:canary

Conversation

@bernabedev
Copy link
Contributor

Formbricks template

Adds a Formbricks template with Docker Compose support and initial environment setup, making it easier to deploy and configure the application.

image

Copy link
Contributor

@DJKnaeckebrot DJKnaeckebrot left a comment

Choose a reason for hiding this comment

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

Good work so far!

I have modified it specially regarding the ports to match it to dokploys standard :)


return {
envs,
mounts,
Copy link
Contributor

Choose a reason for hiding this comment

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

If you don't need it you might wanna get rid of it

depends_on:
- postgres
ports:
- ${PORT}:3000
Copy link
Contributor

Choose a reason for hiding this comment

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

This will lead to port conflics. Make it
ports:
- 3000

const domains: DomainSchema[] = [
{
host: mainDomain,
port: 8080,
Copy link
Contributor

Choose a reason for hiding this comment

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

Set this to 3000, as the cotainer defaults exposes that

Copy link
Contributor

@Siumauricio Siumauricio left a comment

Choose a reason for hiding this comment

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

Nice

@Siumauricio Siumauricio merged commit 18c6d08 into Dokploy:canary Jan 31, 2025
2 checks passed
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.

3 participants