[PR #806] Add SMTP_PORT environment variable to docker-compose #4278

New Issue

GiteaMirror · 2026-04-30T06:51:16-05:00

GiteaMirror commented

2026-04-30 06:51:16 -05:00

📋 Pull Request Information

Original PR: https://github.com/Dokploy/templates/pull/806
Author: @shgxz
Created: 4/18/2026
Status: 🔄 Open

Base: main ← Head: patch-1

📝 Commits (1)

b5d7850 Add SMTP_PORT environment variable to docker-compose

📊 Changes

1 file changed (+3 additions, -1 deletions)

View changed files

📝 blueprints/typebot/docker-compose.yml (+3 -1)

📄 Description

What is this PR about?

Due to SMTP_PORT not mentioned in docker compose file, emails were not being sent.

New PR of [Template Name]

Checklist

Before submitting this PR, please make sure that:

I have read the suggestions in the README.md file https://github.com/Dokploy/templates?tab=readme-ov-file#general-requirements-when-creating-a-template
I have tested the template in my instance, so the maintainers don't spend time trying to figure out what's wrong.
I have added tests that demonstrate that my correction works or that my new feature works.

Close automatically the related issues using the keywords: closes #ISSUE_NUMBER

Screenshots or Videos

Greptile Summary

Adds the missing SMTP_PORT environment variable to both typebot-builder and typebot-viewer services in the Typebot docker-compose. The variable was already declared in template.toml with a default value of "25", so this change correctly closes the gap that was preventing emails from being sent. The only minor concern is that SMTP_PORT: ${SMTP_PORT} is unquoted while every other env var in the file uses single-quoted substitution syntax.

Confidence Score: 5/5

Safe to merge — the fix is correct and the only findings are minor style inconsistencies.

The change correctly adds the missing SMTP_PORT variable to both services, matching the existing template.toml definition. All remaining comments are P2 style suggestions about quoting consistency that don't affect runtime behavior.

No files require special attention.

_{Reviews (1): Last reviewed commit: "Add SMTP_PORT environment variable to do..." | Re-trigger Greptile}

Greptile also left 2 inline comments on this PR.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Dokploy/templates/pull/806 **Author:** [@shgxz](https://github.com/shgxz) **Created:** 4/18/2026 **Status:** 🔄 Open **Base:** `main` ← **Head:** `patch-1` --- ### 📝 Commits (1) - [`b5d7850`](https://github.com/Dokploy/templates/commit/b5d78508556f4a2c6b37eea08a10f448f6845a82) Add SMTP_PORT environment variable to docker-compose ### 📊 Changes **1 file changed** (+3 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `blueprints/typebot/docker-compose.yml` (+3 -1) </details> ### 📄 Description ## What is this PR about? Due to SMTP_PORT not mentioned in docker compose file, emails were not being sent. New PR of [Template Name] ## Checklist Before submitting this PR, please make sure that: - [ ] I have read the suggestions in the README.md file https://github.com/Dokploy/templates?tab=readme-ov-file#general-requirements-when-creating-a-template - [ ] I have tested the template in my instance, so the maintainers don't spend time trying to figure out what's wrong. - [ ] I have added tests that demonstrate that my correction works or that my new feature works. ## Issues related (if applicable) Close automatically the related issues using the keywords: `closes #ISSUE_NUMBER` ## Screenshots or Videos  <h3>Greptile Summary</h3> Adds the missing `SMTP_PORT` environment variable to both `typebot-builder` and `typebot-viewer` services in the Typebot docker-compose. The variable was already declared in `template.toml` with a default value of `"25"`, so this change correctly closes the gap that was preventing emails from being sent. The only minor concern is that `SMTP_PORT: ${SMTP_PORT}` is unquoted while every other env var in the file uses single-quoted substitution syntax. <h3>Confidence Score: 5/5</h3> Safe to merge — the fix is correct and the only findings are minor style inconsistencies. The change correctly adds the missing SMTP_PORT variable to both services, matching the existing template.toml definition. All remaining comments are P2 style suggestions about quoting consistency that don't affect runtime behavior. No files require special attention. <sub>Reviews (1): Last reviewed commit: ["Add SMTP\_PORT environment variable to do..."](https://github.com/dokploy/templates/commit/b5d78508556f4a2c6b37eea08a10f448f6845a82) | [Re-trigger Greptile](https://app.greptile.com/api/retrigger?id=28869828)</sub> > Greptile also left **2 inline comments** on this PR.  --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

GiteaMirror added the pull-request label 2026-04-30 06:51:16 -05:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/templates#4278