close
Skip to content

feat: add OpenToggl LXC container script#1717

Closed
CorrectRoadH wants to merge 3 commits into
community-scripts:mainfrom
CorrectRoadH:feat/opentoggl
Closed

feat: add OpenToggl LXC container script#1717
CorrectRoadH wants to merge 3 commits into
community-scripts:mainfrom
CorrectRoadH:feat/opentoggl

Conversation

@CorrectRoadH
Copy link
Copy Markdown

@CorrectRoadH CorrectRoadH commented Apr 19, 2026 •
edited by CrazyWolf13
Loading

Scripts which are clearly AI generated and not further revised by the Author of this PR (in terms of Coding Standards and Script Layout) may be closed without review.

✍️ Description

Adds a new CT script for OpenToggl, an open-source self-hosted time-tracking application.

The install script deploys the pre-compiled single-file binary from GitHub Releases, sets up PostgreSQL 16 and Redis as local dependencies, writes /opt/opentoggl/.env, and registers a systemd service that runs opentoggl serve on port 8080.

🔗 Related PR / Issue

Link: #

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No breaking changes – Existing functionality remains intact.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix – Resolves an issue without breaking functionality.
  • ✨ New feature – Adds new, non-breaking functionality.
  • 💥 Breaking change – Alters existing functionality in a way that may require updates.
  • 🆕 New script – A fully functional and tested script or script set.
  • 🌍 Website update – Changes to website-related JSON files or metadata.
  • 🔧 Refactoring / Code Cleanup – Improves readability or maintainability without changing functionality.
  • 📝 Documentation update – Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

🔍 Code & Security Review (X in brackets)

  • Follows Code_Audit.md & CONTRIBUTING.md guidelines
  • Uses correct script structure (AppName.sh, AppName-install.sh, AppName.json)
  • No hardcoded credentials

📋 Additional Information (optional)

  • Uses fetch_and_deploy_gh_release in singlefile mode for the prebuilt binary
  • Uses setup_postgresql / setup_postgresql_db helpers for the database
  • Update path uses check_for_gh_release to stop the service, redeploy the binary, and restart

📦 Application Requirements (for new scripts)

Required for 🆕 New script submissions.
Pull requests that do not meet these requirements may be closed without review.

  • The application is at least 6 months old
  • The application is actively maintained
  • The application has 600+ GitHub stars
  • Official release tarballs are published
  • I understand that not all scripts will be accepted due to various reasons and criteria by the community-scripts ORG

🌐 Source

https://github.com/CorrectRoadH/opentoggl

@CorrectRoadH @claude
feat(opentoggl): add OpenToggl LXC container script
80ebb9c 
New open-source time-tracking application using a pre-compiled binary
from GitHub Releases. Requires PostgreSQL 16 + Redis; runs on port 8080.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@CorrectRoadH CorrectRoadH requested a review from a team as a code owner April 19, 2026 04:35
@github-actions github-actions Bot closed this Apr 19, 2026
MickLesk
MickLesk approved these changes Apr 19, 2026
View reviewed changes
Comment thread install/opentoggl-install.sh Outdated
Comment thread install/opentoggl-install.sh Outdated
@MickLesk MickLesk reopened this Apr 19, 2026
@CorrectRoadH
fix(opentoggl): address review feedback
cde1657 
- Drop get_lxc_ip call (already part of core build)
- Combine redis + opentoggl systemctl enable into a single line
@github-actions github-actions Bot closed this Apr 19, 2026
@CorrectRoadH
Copy link
Copy Markdown
Author

CorrectRoadH commented Apr 19, 2026 •
edited
Loading

Thanks for the review @MickLesk! Both suggestions applied.

I just noticed the "6 months old" and "600+ stars" requirements — OpenToggl doesn't meet them yet, so I will re-open once it does. Thanks! 🙏

@community-scripts community-scripts deleted a comment from github-actions Bot Apr 19, 2026
@community-scripts community-scripts deleted a comment from github-actions Bot Apr 19, 2026
@MickLesk MickLesk reopened this Apr 19, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 19, 2026

❌ Pull Request Closed – Application Requirements Not Met

This pull request is marked as 🆕 New script, but the required application criteria were not confirmed.

The following requirement confirmations are missing:

  • The application is at least 6 months old
  • The application is actively maintained
  • The application has 600+ GitHub stars
  • Official release tarballs are published
  • I understand that not all scripts will be accepted

New application submissions must meet the project requirements before being considered.
Please wait until the application satisfies the criteria before submitting a new PR.

⚠ Maintainer note

The team periodically reviews closed submissions. If a project is still considered valuable to the ecosystem, maintainers may reopen the PR even if it does not fully meet the thresholds.

Please do not ping or repeatedly contact maintainers to reopen PRs.

@github-actions github-actions Bot closed this Apr 19, 2026
@MickLesk
Copy link
Copy Markdown
Member

MickLesk commented Apr 19, 2026

@CrazyWolf13 your bot went crazy?

@MickLesk
Copy link
Copy Markdown
Member

MickLesk commented Apr 19, 2026

Ah, you use the PR Template from VE Not from VED. VED has more checkboxes, If they are Not filled -> bot close

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 19, 2026

❌ Pull Request Closed – Application Requirements Not Met

This pull request is marked as 🆕 New script, but the required application criteria were not confirmed.

The following requirement confirmations are missing:

  • The application is at least 6 months old
  • The application has 600+ GitHub stars

New application submissions must meet the project requirements before being considered.
Please wait until the application satisfies the criteria before submitting a new PR.

⚠ Maintainer note

The team periodically reviews closed submissions. If a project is still considered valuable to the ecosystem, maintainers may reopen the PR even if it does not fully meet the thresholds.

Please do not ping or repeatedly contact maintainers to reopen PRs.

@CorrectRoadH
Copy link
Copy Markdown
Author

CorrectRoadH commented Apr 19, 2026 •
edited
Loading

Updated the description to use the correct ProxmoxVED PR template with all checkboxes filled. I'm unable to reopen the PR myself — could a maintainer please reopen? Thanks @MickLesk!

@CorrectRoadH CorrectRoadH changed the title feat(opentoggl): add OpenToggl LXC container script feat: add OpenToggl LXC container script Apr 19, 2026
@MickLesk MickLesk reopened this Apr 19, 2026
@CrazyWolf13
Copy link
Copy Markdown
Member

CrazyWolf13 commented Apr 19, 2026

No, my bot did not go crazy, but the whole section was missing. I handled this case seperately now.

@MickLesk Flow should be fixed now. you can also add keep-open label to keep to PR open

@MickLesk
Copy link
Copy Markdown
Member

MickLesk commented Jun 1, 2026

Repo is disabled / Deleted

@MickLesk MickLesk closed this Jun 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MickLesk MickLesk MickLesk approved these changes

Assignees

No one assigned

Labels

keep-open new script

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@CorrectRoadH @MickLesk @CrazyWolf13