Skip to content

Conversation

dellthePROgrammer
Copy link
Contributor

@dellthePROgrammer dellthePROgrammer commented Sep 5, 2025

πŸ›‘ New scripts must first be submitted to ProxmoxVED for testing.
PRs for new scripts that skip this process will be closed.


✍️ Description

πŸ”— Related PR / Issue

Link: #

βœ… Prerequisites (X in brackets)

  • [X] Self-review completed – Code follows project standards.
  • [X] Tested thoroughly – Changes work as expected.
  • [X] No breaking changes – Existing functionality remains intact.
  • [X] 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.
  • [X] ✨ New feature – Adds new, non-breaking functionality.
  • πŸ’₯ Breaking change – Alters existing functionality in a way that may require updates.
  • [X] πŸ†• 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)

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

πŸ“‹ Additional Information (optional)

Add 4 new helper scripts and update config to help with cloudflare defaults which are commented out by default

addsite - creates a config file with the parrameters you provide
dissite - deletes the symlink for the config file
ensite - creates a symlink for the config file
editsite - opens nano text editor for the site config

please ensure that the link in the traefik.sh file points to the correct repo for the install-traefik.sh

@dellthePROgrammer dellthePROgrammer requested a review from a team as a code owner September 5, 2025 15:10
@dellthePROgrammer
Copy link
Contributor Author

worked in my pull just need to merge and test with VED repo and ensure links are correct

@MickLesk
Copy link
Member

MickLesk commented Sep 6, 2025

@tremor021 your Part? Its the traefik fix, idk the latest conversation πŸ˜„

@CrazyWolf13 CrazyWolf13 changed the title Add Helper Scripts Add Helper Scripts for traefik Sep 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants