martin/ProxmoxVE
1
0
Fork 0
mirror of https://github.com/community-scripts/ProxmoxVE synced 2025-02-12 02:39:17 +00:00
Code Issues Actions 8 Packages Projects Releases Wiki Activity

Check default settings availability in DefaultSettings component and handle undefined values more gracefully in rendering

Browse Source
This commit is contained in:
Bram Suurd 2024-11-06 19:51:52 +01:00
parent 994e958df0
commit db82fe8ec6
1 changed files with 9 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
frontend/src/app/scripts/_components/ScriptItems/DefaultSettings.tsx
View File

@ -5,23 +5,28 @@ export default function DefaultSettings({ item }: { item: Script }) {
(method) => method.type === "default", (method) => method.type === "default",
); );
const defaultSettingsAvailable =
defaultSettings?.resources.cpu !== null ||
defaultSettings?.resources.ram !== null ||
defaultSettings?.resources.hdd !== null;
const defaultAlpineSettings = item.install_methods.find( const defaultAlpineSettings = item.install_methods.find(
(method) => method.type === "alpine", (method) => method.type === "alpine",
); );
return ( return (
<> <>
{defaultSettings && ( {defaultSettingsAvailable && (
<div> <div>
<h2 className="text-md font-semibold">Default settings</h2> <h2 className="text-md font-semibold">Default settings</h2>
<p className="text-sm text-muted-foreground"> <p className="text-sm text-muted-foreground">
CPU: {defaultSettings.resources.cpu}vCPU CPU: {defaultSettings?.resources.cpu}vCPU
</p> </p>
<p className="text-sm text-muted-foreground"> <p className="text-sm text-muted-foreground">
RAM: {defaultSettings.resources.ram}MB RAM: {defaultSettings?.resources.ram}MB
</p> </p>
<p className="text-sm text-muted-foreground"> <p className="text-sm text-muted-foreground">
HDD: {defaultSettings.resources.hdd}GB HDD: {defaultSettings?.resources.hdd}GB
</p> </p>
</div> </div>
)} )}