[PR #686] [CLOSED] Update Minepanel to version 1.7.3 in meta.json and docker-compose.yml… #9248

Closed
opened 2026-06-02 06:06:13 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/Dokploy/templates/pull/686
Author: @Ketbome
Created: 1/29/2026
Status: ❌ Closed

Base: canary ← Head: minepanel-update


📝 Commits (2)

  • 5bb25bb Update Minepanel to version 1.7.3 in meta.json and docker-compose.yml, changing website and documentation links to reflect the new domain.
  • 15a04ea Refactor Minepanel docker-compose.yml to use environment variables for volume paths

📊 Changes

2 files changed (+7 additions, -11 deletions)

View changed files

📝 blueprints/minepanel/docker-compose.yml (+4 -8)
📝 meta.json (+3 -3)

📄 Description

What is this PR about?

Update Minepanel template to version 1.7.3

Changes:

  • Updated backend image: ketbom/minepanel-backend:1.7.1 → 1.7.3
  • Updated frontend image: ketbom/minepanel-frontend:1.7.1 → 1.7.3
  • Updated website and docs links from ketbome.lat → ketbome.com

Checklist

Before submitting this PR, please make sure that:

  • I have read the suggestions in the README.md file
  • I have tested the template in my instance
  • I have added tests that demonstrate that my correction works or that my new feature works.

Issues related (if applicable)

N/A

Screenshots or Videos

image image

Greptile Summary

Updated Minepanel template from version 1.7.1 to 1.7.3, updated domain links from ketbome.lat to ketbome.com, and attempted to refactor volume mounts from named volumes to environment variable-based bind mounts.

Critical Issue Found:

  • The docker-compose.yml references ${DATA_DIR} on line 15, but this variable is not defined in template.toml. This will cause the deployment to fail when the data volume mount is undefined.

Options to fix:

  1. Add DATA_DIR to the [config.env] section in template.toml with an appropriate value (e.g., DATA_DIR = "/app/data")
  2. Revert the volume changes back to named volumes (minepanel-data) and restore the volumes: section

The version updates and domain changes in meta.json are correct.

Confidence Score: 1/5

  • This PR has a critical bug that will cause deployment failure
  • The docker-compose.yml references an undefined environment variable ${DATA_DIR} which will cause the volume mount to fail. This is a deployment-breaking issue that must be fixed before merge.
  • blueprints/minepanel/docker-compose.yml requires immediate attention - missing DATA_DIR definition

Last reviewed commit: 15a04ea

(5/5) You can turn off certain types of comments like style here!


🔄 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/686 **Author:** [@Ketbome](https://github.com/Ketbome) **Created:** 1/29/2026 **Status:** ❌ Closed **Base:** `canary` ← **Head:** `minepanel-update` --- ### 📝 Commits (2) - [`5bb25bb`](https://github.com/Dokploy/templates/commit/5bb25bbe67ba06a045b0c4cbb0573dc22ef8d831) Update Minepanel to version 1.7.3 in meta.json and docker-compose.yml, changing website and documentation links to reflect the new domain. - [`15a04ea`](https://github.com/Dokploy/templates/commit/15a04eadac34f7535eae78f9b619ac872d45ab39) Refactor Minepanel docker-compose.yml to use environment variables for volume paths ### 📊 Changes **2 files changed** (+7 additions, -11 deletions) <details> <summary>View changed files</summary> 📝 `blueprints/minepanel/docker-compose.yml` (+4 -8) 📝 `meta.json` (+3 -3) </details> ### 📄 Description ## What is this PR about? Update Minepanel template to version 1.7.3 ### Changes: - Updated backend image: `ketbom/minepanel-backend:1.7.1` → `1.7.3` - Updated frontend image: `ketbom/minepanel-frontend:1.7.1` → `1.7.3` - Updated website and docs links from `ketbome.lat` → `ketbome.com` ## Checklist Before submitting this PR, please make sure that: - [x] I have read the suggestions in the README.md file - [x] I have tested the template in my instance - [x] I have added tests that demonstrate that my correction works or that my new feature works. ## Issues related (if applicable) N/A ## Screenshots or Videos <img width="1232" height="630" alt="image" src="https://github.com/user-attachments/assets/ecce6e67-04a8-4e6f-9baf-0c468f9ecefc" /> <img width="1864" height="918" alt="image" src="https://github.com/user-attachments/assets/a9f38f29-732d-4d25-b614-b705c8006036" /> <!-- greptile_comment --> <h3>Greptile Summary</h3> Updated Minepanel template from version 1.7.1 to 1.7.3, updated domain links from `ketbome.lat` to `ketbome.com`, and attempted to refactor volume mounts from named volumes to environment variable-based bind mounts. **Critical Issue Found:** - The `docker-compose.yml` references `${DATA_DIR}` on line 15, but this variable is not defined in `template.toml`. This will cause the deployment to fail when the data volume mount is undefined. **Options to fix:** 1. Add `DATA_DIR` to the `[config.env]` section in `template.toml` with an appropriate value (e.g., `DATA_DIR = "/app/data"`) 2. Revert the volume changes back to named volumes (`minepanel-data`) and restore the `volumes:` section The version updates and domain changes in `meta.json` are correct. <h3>Confidence Score: 1/5</h3> - This PR has a critical bug that will cause deployment failure - The `docker-compose.yml` references an undefined environment variable `${DATA_DIR}` which will cause the volume mount to fail. This is a deployment-breaking issue that must be fixed before merge. - `blueprints/minepanel/docker-compose.yml` requires immediate attention - missing `DATA_DIR` definition <sub>Last reviewed commit: 15a04ea</sub> <!-- greptile_other_comments_section --> <sub>(5/5) You can turn off certain types of comments like style [here](https://app.greptile.com/review/github)!</sub> <!-- /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-02 06:06:13 -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#9248