[PR #1719] [MERGED] [Core] Better Creation of App Headers for next feature #3181

Open
opened 2025-11-20 05:33:50 -05:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/1719
Author: @MickLesk
Created: 1/24/2025
Status: Merged
Merged: 1/24/2025
Merged by: @MickLesk

Base: mainHead: app_headers


📝 Commits (2)

  • 08b4901 Update Function to crawl app Headers
  • ae92ec8 Update Function to crawl app Headers

📊 Changes

2 files changed (+30 additions, -39 deletions)

View changed files

📝 .github/workflows/auto-update-app-headers.yml (+14 -18)
📝 .github/workflows/scripts/generate-app-headers.sh (+16 -21)

📄 Description

✍️ Description


  • Related Issue: #
  • Related PR: #
  • Related Discussion: #

Prerequisites

The following steps must be completed for the pull request to be considered:

  • Self-review performed (I have reviewed my code to ensure it follows established patterns and conventions.)
  • Testing performed (I have thoroughly tested my changes and verified expected functionality.)

🛠️ Type of Change

Please check the relevant options:

  • Bug fix (non-breaking change that resolves an issue)
  • New feature (non-breaking change that adds functionality)
  • [] Breaking change (fix or feature that would cause existing functionality to change unexpectedly)
  • [] New script (a fully functional and thoroughly tested script or set of scripts)

📋 Additional Information (optional)

Provide any extra context or screenshots about the feature or fix 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/community-scripts/ProxmoxVE/pull/1719 **Author:** [@MickLesk](https://github.com/MickLesk) **Created:** 1/24/2025 **Status:** ✅ Merged **Merged:** 1/24/2025 **Merged by:** [@MickLesk](https://github.com/MickLesk) **Base:** `main` ← **Head:** `app_headers` --- ### 📝 Commits (2) - [`08b4901`](https://github.com/community-scripts/ProxmoxVE/commit/08b490142c030c0f63069425f785b14b915be1da) Update Function to crawl app Headers - [`ae92ec8`](https://github.com/community-scripts/ProxmoxVE/commit/ae92ec85bbd79458c1eba7b924e412029b44a2bb) Update Function to crawl app Headers ### 📊 Changes **2 files changed** (+30 additions, -39 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/auto-update-app-headers.yml` (+14 -18) 📝 `.github/workflows/scripts/generate-app-headers.sh` (+16 -21) </details> ### 📄 Description ## ✍️ Description - - - - Related Issue: # - Related PR: # - Related Discussion: # - - - ## ✅ Prerequisites The following steps must be completed for the pull request to be considered: - [x] Self-review performed (I have reviewed my code to ensure it follows established patterns and conventions.) - [x] Testing performed (I have thoroughly tested my changes and verified expected functionality.) ## 🛠️ Type of Change Please check the relevant options: - [x] Bug fix (non-breaking change that resolves an issue) - [x] New feature (non-breaking change that adds functionality) - [] Breaking change (fix or feature that would cause existing functionality to change unexpectedly) - [] New script (a fully functional and thoroughly tested script or set of scripts) --- ## 📋 Additional Information (optional) Provide any extra context or screenshots about the feature or fix here. --- <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:33:50 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/ProxmoxVE#3181