From 737aeb422723dcecb73a7fc847c5910333186042 Mon Sep 17 00:00:00 2001 From: Darshan Date: Fri, 23 May 2025 12:50:32 +0530 Subject: [PATCH 1/4] update: fix status code with correct options, show target to rules. --- src/lib/stores/domains.ts | 14 ++++++-------- .../function-[function]/domains/table.svelte | 14 +++++++++----- .../sites/site-[site]/domains/table.svelte | 14 +++++++++----- 3 files changed, 24 insertions(+), 18 deletions(-) diff --git a/src/lib/stores/domains.ts b/src/lib/stores/domains.ts index ea339f83d..348e10c9b 100644 --- a/src/lib/stores/domains.ts +++ b/src/lib/stores/domains.ts @@ -1,23 +1,21 @@ +import { StatusCode } from '@appwrite.io/console'; + export const statusCodeOptions = [ { label: '301 Moved permanently', - value: 301 + value: StatusCode.MovedPermanently301 }, { label: '302 Found', - value: 302 - }, - { - label: '303 See other', - value: 303 + value: StatusCode.Found302 }, { label: '307 Temporary redirect', - value: 307 + value: StatusCode.TemporaryRedirect307 }, { label: '308 Permanent redirect', - value: 308 + value: StatusCode.PermanentRedirect308 } ]; diff --git a/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/table.svelte b/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/table.svelte index 65813ed6a..4cd12187b 100644 --- a/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/table.svelte +++ b/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/table.svelte @@ -27,6 +27,14 @@ let showDelete = $state(false); let showRetry = $state(false); let selectedDomain: Models.ProxyRule = $state(null); + + const domainTarget = (proxy: Models.ProxyRule) => { + return proxy?.redirectUrl + ? 'Redirect to ' + proxy.redirectUrl + : proxy?.deploymentVcsProviderBranch + ? 'Deployed from' + proxy.deploymentVcsProviderBranch + : 'Active deployment'; + }; @@ -64,11 +72,7 @@ {/if} {:else if column.id === 'target'} - {domain?.redirectUrl - ? 'Redirect to ' + domain.redirectUrl - : domain?.deploymentVcsProviderBranch - ? 'Deployed from' + domain.deploymentVcsProviderBranch - : '-'} + {domainTarget(domain)} {/if} {/each} diff --git a/src/routes/(console)/project-[region]-[project]/sites/site-[site]/domains/table.svelte b/src/routes/(console)/project-[region]-[project]/sites/site-[site]/domains/table.svelte index 020947970..0d82b3bc3 100644 --- a/src/routes/(console)/project-[region]-[project]/sites/site-[site]/domains/table.svelte +++ b/src/routes/(console)/project-[region]-[project]/sites/site-[site]/domains/table.svelte @@ -27,6 +27,14 @@ let showDelete = $state(false); let showRetry = $state(false); let selectedDomain: Models.ProxyRule = $state(null); + + const domainTarget = (proxy: Models.ProxyRule) => { + return proxy?.redirectUrl + ? 'Redirect to ' + proxy.redirectUrl + : proxy?.deploymentVcsProviderBranch + ? 'Deployed from' + proxy.deploymentVcsProviderBranch + : 'Active deployment'; + }; @@ -65,11 +73,7 @@ {/if} {:else if column.id === 'target'} - {domain?.redirectUrl - ? 'Redirect to ' + domain.redirectUrl - : domain?.deploymentVcsProviderBranch - ? 'Deployed from' + domain.deploymentVcsProviderBranch - : '-'} + {domainTarget(domain)} {/if} {/each} From 289d9fc94c216fcf4469e583aac348ce2919ba4f Mon Sep 17 00:00:00 2001 From: Darshan Date: Fri, 23 May 2025 13:04:30 +0530 Subject: [PATCH 2/4] update: rename all `domain` vars to `proxy` if `ProxyRule` type for clarity in functions. --- .../function-[function]/domains/+page.svelte | 6 ++-- .../function-[function]/domains/+page.ts | 2 +- .../domains/deleteDomainModal.svelte | 11 +++---- .../domains/recordsCard.svelte | 7 ++--- .../domains/retryDomainModal.svelte | 15 +++++----- .../function-[function]/domains/table.svelte | 30 +++++++++---------- 6 files changed, 35 insertions(+), 36 deletions(-) diff --git a/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/+page.svelte b/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/+page.svelte index e1ab5a837..eaba6f04b 100644 --- a/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/+page.svelte +++ b/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/+page.svelte @@ -18,7 +18,7 @@ let showDelete = $state(false); let showRetry = $state(false); - let selectedDomain: Models.ProxyRule = null; + let selectedProxyRule: Models.ProxyRule = null; @@ -83,9 +83,9 @@ {#if showDelete} - + {/if} {#if showRetry} - + {/if} diff --git a/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/+page.ts b/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/+page.ts index 8e96e426a..50397306c 100644 --- a/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/+page.ts +++ b/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/+page.ts @@ -26,7 +26,7 @@ export const load: PageLoad = async ({ depends, params, url, route }) => { limit, query, search, - domains: await sdk + proxyRules: await sdk .forProject(params.region, params.project) .proxy.listRules( [ diff --git a/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/deleteDomainModal.svelte b/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/deleteDomainModal.svelte index 1882bc282..c5ce1db80 100644 --- a/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/deleteDomainModal.svelte +++ b/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/deleteDomainModal.svelte @@ -11,15 +11,16 @@ import { page } from '$app/state'; export let show = false; - export let selectedDomain: Models.ProxyRule; - let confirm = false; + export let selectedProxyRule: Models.ProxyRule; + let error = ''; + let confirm = false; async function deleteDomain() { try { await sdk .forProject(page.params.region, page.params.project) - .proxy.deleteRule(selectedDomain.$id); + .proxy.deleteRule(selectedProxyRule.$id); await invalidate(Dependencies.FUNCTION_DOMAINS); show = false; addNotification({ @@ -35,13 +36,13 @@ - {#if selectedDomain} + {#if selectedProxyRule} Are you sure you want to delete this domain? You will no longer be able to execute your function by visiting: - {selectedDomain.domain} + {selectedProxyRule.domain} {/if} import { Link } from '$lib/elements'; - import type { Domain } from '$lib/sdk/domains'; import { IconInfo } from '@appwrite.io/pink-icons-svelte'; import { Badge, @@ -14,7 +13,7 @@ import type { Models } from '@appwrite.io/console'; import { consoleVariables } from '$routes/(console)/store'; - export let domain: Domain | Models.ProxyRule; + export let proxyRule: Models.ProxyRule;
@@ -22,7 +21,7 @@ - {domain?.domain} + {proxyRule?.domain} @@ -39,7 +38,7 @@ CNAME - {domain?.domain} + {proxyRule?.domain} - - {#if selectedDomain} - + + {#if selectedProxyRule} + {/if} diff --git a/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/table.svelte b/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/table.svelte index 4cd12187b..1c9df426f 100644 --- a/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/table.svelte +++ b/src/routes/(console)/project-[region]-[project]/functions/function-[function]/domains/table.svelte @@ -19,16 +19,16 @@ import { columns } from './store'; let { - domains + proxyRules }: { - domains: Models.ProxyRuleList; + proxyRules: Models.ProxyRuleList; } = $props(); let showDelete = $state(false); let showRetry = $state(false); - let selectedDomain: Models.ProxyRule = $state(null); + let selectedProxyRule: Models.ProxyRule = $state(null); - const domainTarget = (proxy: Models.ProxyRule) => { + const proxyTarget = (proxy: Models.ProxyRule) => { return proxy?.redirectUrl ? 'Redirect to ' + proxy.redirectUrl : proxy?.deploymentVcsProviderBranch @@ -46,24 +46,24 @@ {/each} - {#each domains.rules as domain} + {#each proxyRules.rules as proxyRule} {#each $columns as column} {#if column.id === 'domain'} - + - {domain.domain} + {proxyRule.domain} - {#if domain.status === 'verifying'} + {#if proxyRule.status === 'verifying'} - {:else if domain.status !== 'verified'} + {:else if proxyRule.status !== 'verified'} {:else if column.id === 'target'} - {domainTarget(domain)} + {proxyTarget(proxyRule)} {/if} {/each} @@ -91,11 +91,11 @@ - {#if domain.status !== 'verified'} + {#if proxyRule.status !== 'verified'} { - selectedDomain = domain; + selectedProxyRule = proxyRule; showRetry = true; toggle(e); }}> @@ -106,7 +106,7 @@ status="danger" leadingIcon={IconTrash} on:click={(e) => { - selectedDomain = domain; + selectedProxyRule = proxyRule; showDelete = true; toggle(e); trackEvent(Click.DomainDeleteClick, { @@ -125,9 +125,9 @@ {#if showDelete} - + {/if} {#if showRetry} - + {/if} From 6ead88a74353d58e545675da6fc36269caa75da2 Mon Sep 17 00:00:00 2001 From: Darshan Date: Fri, 23 May 2025 13:50:25 +0530 Subject: [PATCH 3/4] update: rename all `domain` vars to `proxy` if `ProxyRule` type for clarity in sites. --- .../sites/site-[site]/domains/+page.svelte | 6 ++-- .../sites/site-[site]/domains/+page.ts | 2 +- .../domains/deleteDomainModal.svelte | 10 +++---- .../domains/retryDomainModal.svelte | 14 ++++----- .../sites/site-[site]/domains/table.svelte | 30 +++++++++---------- 5 files changed, 31 insertions(+), 31 deletions(-) diff --git a/src/routes/(console)/project-[region]-[project]/sites/site-[site]/domains/+page.svelte b/src/routes/(console)/project-[region]-[project]/sites/site-[site]/domains/+page.svelte index 23537ff99..8e4d4c3e7 100644 --- a/src/routes/(console)/project-[region]-[project]/sites/site-[site]/domains/+page.svelte +++ b/src/routes/(console)/project-[region]-[project]/sites/site-[site]/domains/+page.svelte @@ -29,14 +29,14 @@ - {#if data.domains.total} - + {#if data.proxyRules.total} +
+ total={data.proxyRules.total} /> {:else if data?.search} - {#if data.domains.total} -
+ {#if data.proxyRules.total} +
+ total={data.proxyRules.total} /> {:else if data?.search}