[PR #20] [MERGED] Add a discussion template for script requests #2373

Closed
opened 2025-11-20 05:29:31 -05:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/20
Author: @havardthom
Created: 11/2/2024
Status: Merged
Merged: 11/2/2024
Merged by: @tteck

Base: mainHead: main


📝 Commits (1)

  • cb1df81 Add a discussion template for script requests

📊 Changes

2 files changed (+36 additions, -1 deletions)

View changed files

.github/DISCUSSION_TEMPLATE/request-script.yml (+35 -0)
📝 .github/pull_request_template.md (+1 -1)

📄 Description

Note

I am meticulous when it comes to merging code into the main branch, so please understand that I may reject pull requests that do not meet the project's standards. It's never personal. Also, game-related scripts have a lower chance of being merged.

Description

Fixes https://github.com/community-scripts/ProxmoxVE/discussions/6

Add template for script requests in discussions. New discussion category "Request script" must be added by a repo owner first (see third screenshot).

Default form:

default_form

Opened discussion:

request_script_open

Create new discussion category:

request_script

Type of change

Please check the relevant option(s):

  • Bug fix (non-breaking change that resolves an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (a fix or feature that would cause existing functionality to change unexpectedly)
  • New script (a fully functional and thoroughly tested script or set of scripts.)
  • Self-review performed (I have reviewed my code, ensuring it follows established patterns and conventions)
  • Documentation update required (this change requires an update to the documentation)

🔄 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/community-scripts/ProxmoxVE/pull/20 **Author:** [@havardthom](https://github.com/havardthom) **Created:** 11/2/2024 **Status:** ✅ Merged **Merged:** 11/2/2024 **Merged by:** [@tteck](https://github.com/tteck) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (1) - [`cb1df81`](https://github.com/community-scripts/ProxmoxVE/commit/cb1df814f0d80681b1a3d919020d740f9bb715c8) Add a discussion template for script requests ### 📊 Changes **2 files changed** (+36 additions, -1 deletions) <details> <summary>View changed files</summary> ➕ `.github/DISCUSSION_TEMPLATE/request-script.yml` (+35 -0) 📝 `.github/pull_request_template.md` (+1 -1) </details> ### 📄 Description > [!NOTE] I am meticulous when it comes to merging code into the main branch, so please understand that I may reject pull requests that do not meet the project's standards. It's never personal. Also, game-related scripts have a lower chance of being merged. ## Description Fixes https://github.com/community-scripts/ProxmoxVE/discussions/6 Add template for script requests in discussions. **New discussion category "Request script" must be added by a repo owner first (see third screenshot).** Default form: ![default_form](https://github.com/user-attachments/assets/6430d0b1-8209-4623-8772-8e4a57b6f6b5) Opened discussion: ![request_script_open](https://github.com/user-attachments/assets/faafa87d-de63-45e7-84cb-d6d5f719c7cf) Create new discussion category: ![request_script](https://github.com/user-attachments/assets/b6055869-a8eb-48da-8d3e-0ef8093b66a9) ## Type of change Please check the relevant option(s): - [ ] Bug fix (non-breaking change that resolves an issue) - [x] New feature (non-breaking change that adds functionality) - [ ] Breaking change (a fix or feature that would cause existing functionality to change unexpectedly) - [ ] New script (a fully functional and thoroughly tested script or set of scripts.) - [x] Self-review performed (I have reviewed my code, ensuring it follows established patterns and conventions) - [ ] Documentation update required (this change requires an update to the documentation) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
saavagebueno added the pull-request label 2025-11-20 05:29:31 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/ProxmoxVE#2373