Merge pull request #2147 from HarshMN2345/fix-SER-149-Fix-site-settings-saving

Author: ItzNotABug

src/routes/(console)/project-[region]-[project]/sites/site-[site]/settings/+page.svelte

@@ -60,7 +60,10 @@
     {#key data.site.providerRepositoryId}
         <UpdateRepository site={data.site} installations={data.installations} />
     {/key}
-    <UpdateBuildSettings site={data.site} frameworks={data.frameworks.frameworks} />
+    <UpdateBuildSettings
+        site={data.site}
+        frameworks={data.frameworks.frameworks}
+        specs={data.specificationsList} />
     <UpdateRuntimeSettings site={data.site} frameworks={data.frameworks.frameworks} />
     <UpdateVariables
         {sdkCreateVariable}

src/routes/(console)/project-[region]-[project]/sites/site-[site]/settings/+page.ts

@@ -33,6 +33,11 @@ export const load = async ({ params, depends, parent }) => {
         }
     });
 
+    const enabledSpecs = specificationsList?.specifications?.filter((s) => s.enabled) ?? [];
+    if (!enabledSpecs.some((s) => s.slug === site.specification)) {
+        site.specification = enabledSpecs[0]?.slug;
+    }
+
     return {
         site,
         frameworks,

src/routes/(console)/project-[region]-[project]/sites/site-[site]/settings/updateBuildSettings.svelte

@@ -17,10 +17,12 @@
 
     let {
         site,
-        frameworks
+        frameworks,
+        specs
     }: {
         site: Models.Site;
         frameworks: Models.Framework[];
+        specs: Models.SpecificationList;
     } = $props();
 
     let frameworkKey = $state(site.framework);
@@ -88,12 +90,33 @@
         }
     });
 
+    $effect(() => {
+        if (selectedFramework) {
+            if (!selectedFramework.adapters.some((a) => a.key === adapter)) {
+                adapter = selectedFramework.adapters[0].key as Adapter;
+                site.adapter = adapter;
+            }
+            if (specs && specs.specifications?.length) {
+                if (!specs.specifications.some((s) => s.slug === site.specification)) {
+                    site.specification = specs.specifications[0].slug;
+                }
+            }
+        }
+    });
+
     async function update() {
         let adptr = selectedFramework.adapters.find((a) => a.key === adapter);
         if (!adptr?.key && selectedFramework.adapters?.length) {
             adapter = selectedFramework.adapters[0].key as Adapter;
             adptr = selectedFramework.adapters[0];
+            site.adapter = adapter;
         }
+        // only allow enabled specsification for it
+        const enabledSpecs = specs?.specifications?.filter((s) => s.enabled) ?? [];
+        let specToSend = enabledSpecs.some((s) => s.slug === site.specification)
+            ? site.specification
+            : enabledSpecs[0]?.slug;
+        site.specification = specToSend;
         try {
             await sdk
                 .forProject(page.params.region, page.params.project)
@@ -115,7 +138,7 @@
                     site.providerBranch || undefined,
                     site.providerSilentMode || undefined,
                     site.providerRootDirectory || undefined,
-                    site?.specification || undefined
+                    specToSend || undefined
                 );
             await invalidate(Dependencies.SITE);
             addNotification({

src/routes/(console)/project-[region]-[project]/sites/site-[site]/settings/updateResourceLimits.svelte

@@ -19,8 +19,6 @@
     let specification = site.specification;
     let originalSpecification = site.specification;
 
-    $: originalSpecification = site.specification;
-
     async function updateLogging() {
         try {
             await sdk