Skip to content

add angular app - #18302

Closed
Mornante wants to merge 1 commit into
community-scripts:mainfrom
Mornante:mornante
Closed

Mornante wants to merge 1 commit into
community-scripts:mainfrom
Mornante:mornante

Conversation

@Mornante

Copy link
Copy Markdown

✍️ Description

🔗 Related Issue

Fixes #

✅ Prerequisites (x in brackets)

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

Tested on:


🤖 AI Assistance (X in brackets)

If you used an AI tool (GitHub Copilot, Claude, ChatGPT, etc.) to write or generate any scripts in this PR, you must confirm compliance below.
Select exactly one option.

  • No AI used – Scripts were written without AI assistance.
  • AI was used – I confirm the scripts were built using AGENTS.md and .github/agents/pve-script-creator.agent.md as guidance, and the output has been reviewed and corrected to match those guidelines.

🛠️ 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 script metadata (PocketBase/website data).
  • 🔧 Refactoring / Code Cleanup – Improves readability or maintainability without changing functionality.
  • 📝 Documentation update – Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

💥 Breaking Change Advisory (only if you checked "Breaking change")

@Mornante
Mornante requested a review from a team as a code owner October 29, 2026 10:13
@github-actions

Copy link
Copy Markdown
Contributor

👋 Hi @Mornante,

This pull request was closed because it does not follow the PR template.

Please fix the following and open a new PR (or reopen this one after updating the description):

  • Prerequisite not checked: Self-review completed
  • Prerequisite not checked: Tested thoroughly
  • Prerequisite not checked: No security risks
  • At least one Type of Change checkbox must be checked.

Use the template sections, fill in the description, check all prerequisite boxes, and select at least one type of change.

Maintainers can add the keep-open label to exempt a PR from this check.

Thank you for contributing! 🙏

@github-actions github-actions Bot closed this Oct 29, 2026
@github-actions

Copy link
Copy Markdown
Contributor

👋 Hi @Mornante,

Thank you for your interest in contributing a new script!

However, new scripts must first be submitted to our development repository for testing and review before they can be merged here.

🛑 New scripts must be submitted to DevScripts for testing.
PRs without prior testing will be closed.

Please open your PR at https://github.com/community-scripts/DevScripts instead.
Once your script has been tested and approved there, it will be pushed to this repository automatically.

This PR will now be closed. Thank you for understanding! 🙏

@github-actions

Copy link
Copy Markdown
Contributor

Not testable this way yet

mornante-angularapp still uses the older one-liner bootstrap, which
resolves everything from ProxmoxVE/main and ignores COMMUNITY_SCRIPTS_URL.
There is no way to point it at this branch — test it from a checkout on the
host instead, or migrate the script to the _cs_boot bootstrap first.

@github-actions github-actions Bot added not a script issue This is not a script issue new script A change that adds a new script labels Oct 29, 2026
@Mornante
Mornante deleted the mornante branch October 29, 2026 10:14
@Mornante
Mornante restored the mornante branch October 29, 2026 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

missing pr template new script A change that adds a new script not a script issue This is not a script issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant