[PR #746] [CLOSED] feat: add Weblate template #9943

Closed
opened 2026-06-05 09:14:45 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/Dokploy/templates/pull/746
Author: @enigma972
Created: 3/16/2026
Status: Closed

Base: canaryHead: weblate


📝 Commits (9)

  • f2c576d feat: add Weblate template
  • 9bfe810 chore: add db health check
  • 9502f8e feat: cleanup
  • e29916f chore: fix config file
  • 7983f40 chore: remove compose default config
  • 626fdca chore: fix weblate version
  • be412bc chore: run dedupe-and-sort-meta
  • e805d62 Update blueprints/weblate/docker-compose.yml
  • ebb2ba9 chore: remove config.mounts

📊 Changes

4 files changed (+149 additions, -18 deletions)

View changed files

blueprints/weblate/docker-compose.yml (+75 -0)
blueprints/weblate/template.toml (+56 -0)
blueprints/weblate/weblate.svg (+1 -0)
📝 meta.json (+17 -18)

📄 Description

What is this PR about?

New PR of Weblate

Checklist

Before submitting this PR, please make sure that:

Greptile Summary

This PR adds a new Weblate (web-based translation tool) template with Docker Compose services for Weblate, Valkey (Redis-compatible cache), and PostgreSQL 18. The template includes good security practices like read_only containers and tmpfs mounts, proper healthchecks, and correct volume paths for PostgreSQL 18+.

Key issues to address before merging:

  • Self-referencing variables in template.toml: Multiple variables (e.g., POSTGRES_USER, POSTGRES_DB, POSTGRES_HOST, WEBLATE_SITE_TITLE, WEBLATE_SITE_DOMAIN, WEBLATE_ADMIN_NAME) reference themselves instead of providing actual default values, which will cause them to resolve to empty strings and likely break the deployment.
  • Unpinned Docker image: weblate/weblate:latest should be pinned to a specific version (e.g., 5.11.2), and meta.json version field should match.
  • Docker Compose version: Uses 3.9 instead of the project-standard 3.8.

Confidence Score: 2/5

  • This PR has configuration issues that will likely cause deployment failures and should be fixed before merging.
  • Score of 2 reflects that while the overall template structure is sound, the self-referencing variables in template.toml will result in empty/undefined values for critical settings like database credentials and host, which will break the deployment. The unpinned image tag also violates project guidelines.
  • Pay close attention to blueprints/weblate/template.toml (self-referencing variables) and blueprints/weblate/docker-compose.yml (unpinned image tag).

Last reviewed commit: 626fdca

Greptile also left 4 inline comments on this PR.

(2/5) Greptile learns from your feedback when you react with thumbs up/down!

Context used:

  • Rule used - AGENTS.md (source)

🔄 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/746 **Author:** [@enigma972](https://github.com/enigma972) **Created:** 3/16/2026 **Status:** ❌ Closed **Base:** `canary` ← **Head:** `weblate` --- ### 📝 Commits (9) - [`f2c576d`](https://github.com/Dokploy/templates/commit/f2c576d66975160bccf0fa0d8dd95546511fcfdd) feat: add Weblate template - [`9bfe810`](https://github.com/Dokploy/templates/commit/9bfe8105619bffd984e30b31120a527dfb5de6a2) chore: add db health check - [`9502f8e`](https://github.com/Dokploy/templates/commit/9502f8e06663a2597b3eca6e63f6cb17ab4caf27) feat: cleanup - [`e29916f`](https://github.com/Dokploy/templates/commit/e29916f32328dd0f17b341a4f2a51390fac6df07) chore: fix config file - [`7983f40`](https://github.com/Dokploy/templates/commit/7983f40202bd4aeee5cc9baea992929a47fcc03c) chore: remove compose default config - [`626fdca`](https://github.com/Dokploy/templates/commit/626fdca8dfc14b01a18b29e824864cef76e31291) chore: fix weblate version - [`be412bc`](https://github.com/Dokploy/templates/commit/be412bc60553066617a5f1b6c6c4c7b795c1958f) chore: run dedupe-and-sort-meta - [`e805d62`](https://github.com/Dokploy/templates/commit/e805d620fde7fc19d2eb3f02b227d8ca2674db2c) Update blueprints/weblate/docker-compose.yml - [`ebb2ba9`](https://github.com/Dokploy/templates/commit/ebb2ba91a35a8e0d64f991f5a8602a3d693dda9b) chore: remove config.mounts ### 📊 Changes **4 files changed** (+149 additions, -18 deletions) <details> <summary>View changed files</summary> ➕ `blueprints/weblate/docker-compose.yml` (+75 -0) ➕ `blueprints/weblate/template.toml` (+56 -0) ➕ `blueprints/weblate/weblate.svg` (+1 -0) 📝 `meta.json` (+17 -18) </details> ### 📄 Description ## What is this PR about? New PR of [Weblate](https://weblate.org) ## Checklist Before submitting this PR, please make sure that: - [x] 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 - [x] I have tested the template in my instance, so the maintainers don't spend time trying to figure out what's wrong. - [x] I have added tests that demonstrate that my correction works or that my new feature works. <!-- greptile_comment --> <h3>Greptile Summary</h3> This PR adds a new Weblate (web-based translation tool) template with Docker Compose services for Weblate, Valkey (Redis-compatible cache), and PostgreSQL 18. The template includes good security practices like `read_only` containers and `tmpfs` mounts, proper healthchecks, and correct volume paths for PostgreSQL 18+. **Key issues to address before merging:** - **Self-referencing variables in `template.toml`**: Multiple variables (e.g., `POSTGRES_USER`, `POSTGRES_DB`, `POSTGRES_HOST`, `WEBLATE_SITE_TITLE`, `WEBLATE_SITE_DOMAIN`, `WEBLATE_ADMIN_NAME`) reference themselves instead of providing actual default values, which will cause them to resolve to empty strings and likely break the deployment. - **Unpinned Docker image**: `weblate/weblate:latest` should be pinned to a specific version (e.g., `5.11.2`), and `meta.json` `version` field should match. - **Docker Compose version**: Uses `3.9` instead of the project-standard `3.8`. <h3>Confidence Score: 2/5</h3> - This PR has configuration issues that will likely cause deployment failures and should be fixed before merging. - Score of 2 reflects that while the overall template structure is sound, the self-referencing variables in template.toml will result in empty/undefined values for critical settings like database credentials and host, which will break the deployment. The unpinned image tag also violates project guidelines. - Pay close attention to `blueprints/weblate/template.toml` (self-referencing variables) and `blueprints/weblate/docker-compose.yml` (unpinned image tag). <sub>Last reviewed commit: 626fdca</sub> > Greptile also left **4 inline comments** on this PR. <sub>(2/5) Greptile learns from your feedback when you react with thumbs up/down!</sub> **Context used:** - Rule used - AGENTS.md ([source](https://app.greptile.com/review/custom-context?memory=09330bde-2058-497c-9c64-ceae637fb5b2)) <!-- /greptile_comment --> --- <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-06-05 09:14:45 -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#9943