Skip to content

fix: update to postiz templates for v2 with temporal#658

Open
rubixvi wants to merge 6 commits intoDokploy:canaryfrom
rubixvi:postiz
Open

fix: update to postiz templates for v2 with temporal#658
rubixvi wants to merge 6 commits intoDokploy:canaryfrom
rubixvi:postiz

Conversation

@rubixvi
Copy link
Contributor

@rubixvi rubixvi commented Jan 16, 2026

  • Remove deprecated version field for compose file format compliance
  • Add Temporal workflow engine with PostgreSQL and Elasticsearch
  • Configure comprehensive social media platform integrations (Discord, LinkedIn, X, TikTok, YouTube, etc.)
  • Add Cloudflare R2 storage provider configuration options
  • Include email service configuration for notifications
  • Implement Redis persistence and health checks
  • Add proper service networking and dependencies

Temporal and Postiz must be seperated in terms of the databse for it function with the dual database setup.

Greptile Summary

Updated Postiz template to v2 with Temporal workflow engine integration. Added comprehensive social media platform integrations and configuration options for Cloudflare R2 storage and email services.

Key changes:

  • Added Temporal workflow engine with dedicated PostgreSQL and Elasticsearch services
  • Configured dual-database setup separating Postiz and Temporal databases
  • Added social media platform integrations (Discord, LinkedIn, X, TikTok, YouTube, Facebook, etc.)
  • Enhanced Redis with persistence configuration and health checks
  • Added Cloudflare R2 storage provider options
  • Included email service configuration

Issues found:

  • Missing required version: "3.8" field (violates AGENTS.md style guide)
  • Network definitions added (dokploy-network and temporal-network) which may conflict with Dokploy's network isolation model per style guide
  • URL scheme changed from HTTPS to HTTP - verify compatibility with Dokploy's SSL termination

Confidence Score: 2/5

  • This PR has critical style guide violations that must be fixed before merging
  • Score reflects a critical syntax violation (missing version field required by style guide) and network configuration that conflicts with Dokploy conventions. The functional changes appear sound (dual database setup for Temporal separation, comprehensive platform integrations), but the implementation violates documented standards that could cause deployment or isolation issues
  • Pay close attention to blueprints/postiz/docker-compose.yml - must add version field and reconsider network definitions

Last reviewed commit: 21ca847

Context used:

  • Context from dashboard - AGENTS.md (source)

- Remove deprecated version field for compose file format compliance
- Add Temporal workflow engine with PostgreSQL and Elasticsearch
- Configure comprehensive social media platform integrations (Discord, LinkedIn, X, TikTok, YouTube, etc.)
- Add Cloudflare R2 storage provider configuration options
- Include email service configuration for notifications
- Implement Redis persistence and health checks
- Add proper service networking and dependencies
Copilot AI review requested due to automatic review settings January 16, 2026 09:58
@github-actions
Copy link

github-actions bot commented Jan 16, 2026

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
templates ✅ Ready (View Log) Visit Preview 21ca847

@rubixvi
Copy link
Contributor Author

rubixvi commented Jan 16, 2026

@Siumauricio , just so you're aware, I know there's an inclusion of networks, but this is required for the two postgres database to function without conflict.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates the Postiz template to version 2 with Temporal workflow engine integration. The changes add comprehensive social media platform integrations, email and cloud storage configurations, and separate database infrastructure for Temporal.

Changes:

  • Remove deprecated version field from docker-compose.yml for v3.8 compliance
  • Add Temporal workflow engine with dedicated PostgreSQL database and Elasticsearch
  • Configure environment variables for 15+ social media platforms (Discord, LinkedIn, X, TikTok, YouTube, etc.)
  • Add Cloudflare R2 storage and email service configuration options
  • Implement Redis persistence with health checks and explicit networking configuration

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 13 comments.

File Description
blueprints/postiz/docker-compose.yml Adds Temporal services (workflow engine, PostgreSQL, Elasticsearch) with dedicated networking; configures Redis persistence; adds environment variables for social integrations
blueprints/postiz/template.toml Defines environment variables for email, Cloudflare R2 storage, and social media platform API credentials

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.

@SimonSchroffner
Copy link
Contributor

Hi, just wanted to check when this will be done. The current template does not work anymore.

@Siumauricio
Copy link
Contributor

@Siumauricio , just so you're aware, I know there's an inclusion of networks, but this is required for the two postgres database to function without conflict.

Why do we need this?

@Siumauricio
Copy link
Contributor

@greptile review

Copy link

@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.

2 files reviewed, 3 comments

Edit Code Review Agent Settings | Greptile

@@ -1,22 +1,70 @@
version: "3.8"

services:
Copy link

Choose a reason for hiding this comment

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

missing version: "3.8" field - the style guide (AGENTS.md:149) requires Version: MUST be 3.8

Suggested change
services:
version: "3.8"
services:

Context Used: Context from dashboard - AGENTS.md (source)

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Comment on lines +173 to +178
networks:
dokploy-network:
external: true
temporal-network:
driver: bridge
name: temporal-network
Copy link

Choose a reason for hiding this comment

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

defining networks violates the style guide (AGENTS.md:150) which states: "NEVER include: ports (use expose only), container_name, networks (Dokploy handles isolation)"

while some existing templates use dokploy-network for cross-service communication, the separate temporal-network may cause issues with Dokploy's network isolation model

Context Used: Context from dashboard - AGENTS.md (source)

Comment on lines +7 to +9
MAIN_URL: "http://${POSTIZ_HOST}"
FRONTEND_URL: "http://${POSTIZ_HOST}"
NEXT_PUBLIC_BACKEND_URL: "http://${POSTIZ_HOST}/api"
Copy link

Choose a reason for hiding this comment

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

changed URL scheme from https:// to http:// - verify this works correctly with Dokploy's reverse proxy and SSL termination

@rubixvi
Copy link
Contributor Author

rubixvi commented Feb 16, 2026

@Siumauricio , just so you're aware, I know there's an inclusion of networks, but this is required for the two postgres database to function without conflict.

Why do we need this?

Cause the two postgres will end up conflicting with each other. Postiz has one and Temporal has another. And temporal is very demanding compared, logically you can merge the postgres but it'll just degrade performance.

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