Scripts intended for direct LXC install not documented correctly #169

Closed
opened 2026-02-04 16:56:56 +03:00 by OVERLORD · 5 comments
Owner

Originally created by @di5cord20 on GitHub (Dec 13, 2024).

Originally assigned to: @BramSuurdje on GitHub.

Please verify that you have read and understood the guidelines.

yes

What type of issue is this?

Bug

A clear and concise description of the issue.

I noticed that scripts that are meant to be run directly in the LXC console are not stating as such on the web.

Current:
To create a new Proxmox VE VS Code Server , run the command below in the Proxmox VE Shell.

Change to:
To create a new Proxmox VE VS Code Server , run the command below in the LXC console.

Which browser are you using?

Chrome

If relevant, including screenshots or a code block can be helpful in clarifying the issue.

No response

Please provide detailed steps to reproduce the issue.

No response

Originally created by @di5cord20 on GitHub (Dec 13, 2024). Originally assigned to: @BramSuurdje on GitHub. ### Please verify that you have read and understood the guidelines. yes ### What type of issue is this? Bug ### A clear and concise description of the issue. I noticed that scripts that are meant to be run directly in the LXC console are not stating as such on the web. Current: To create a new Proxmox VE VS Code Server , run the command below in the Proxmox VE Shell. Change to: To create a new Proxmox VE VS Code Server , run the command below in the LXC console. ### Which browser are you using? Chrome ### If relevant, including screenshots or a code block can be helpful in clarifying the issue. _No response_ ### Please provide detailed steps to reproduce the issue. _No response_
OVERLORD added the website label 2026-02-04 16:56:56 +03:00
Author
Owner

@MickLesk commented on GitHub (Dec 13, 2024):

Which ones are you talking about? It's in the VS Code. I just did a bigger PR. https://github.com/community-scripts/ProxmoxVE/pull/812

@MickLesk commented on GitHub (Dec 13, 2024): Which ones are you talking about? It's in the VS Code. I just did a bigger PR. https://github.com/community-scripts/ProxmoxVE/pull/812
Author
Owner

@di5cord20 commented on GitHub (Dec 13, 2024):

When I view the website, I see this:
Screenshot from 2024-12-13 08-58-06

Not sure why that happens, as I see the below in the JSON code:
8c1b48453c/json/code-server.json (L35)

This applies to other scripts like File Browser, Tailscale, and likely others.

@di5cord20 commented on GitHub (Dec 13, 2024): When I view the website, I see this: ![Screenshot from 2024-12-13 08-58-06](https://github.com/user-attachments/assets/9b92ea5a-43ec-4b7a-95a3-4e2324816256) Not sure why that happens, as I see the below in the JSON code: https://github.com/community-scripts/ProxmoxVE/blob/8c1b48453ca91b9c9ca43b52db8d2e62eba02b47/json/code-server.json#L35 This applies to other scripts like File Browser, Tailscale, and likely others.
Author
Owner

@MickLesk commented on GitHub (Dec 13, 2024):

That Execute within an existing LXC Console stands above.
image

@BramSuurdje any idea for this "To create a new Proxmox VE VS Code Server , run the command below in the Proxmox VE Shell." ?

@MickLesk commented on GitHub (Dec 13, 2024): That Execute within an existing LXC Console stands above. ![image](https://github.com/user-attachments/assets/5045b688-b55a-4a5a-ad50-6beca41b66e2) @BramSuurdje any idea for this "To create a new Proxmox VE VS Code Server , run the command below in the Proxmox VE Shell." ?
Author
Owner

@BramSuurdje commented on GitHub (Dec 29, 2024):

To fix this would require some additional work. Currently, all it does is check the file path, determine if it is "ct/vm" or "misc," and base the text on that.

@BramSuurdje commented on GitHub (Dec 29, 2024): To fix this would require some additional work. Currently, all it does is check the file path, determine if it is "ct/vm" or "misc," and base the text on that.
Author
Owner

@MickLesk commented on GitHub (Jan 3, 2025):

Actually Not Planned. Ive set it into on Hold and Take a notice about it

@MickLesk commented on GitHub (Jan 3, 2025): Actually Not Planned. Ive set it into on Hold and Take a notice about it
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ProxmoxVE#169