accept multiple whitelabelled domains for sso

nickfixit · Sep 28, 2024 · 56db1d9 · 56db1d9
1 parent 30d9c40
commit 56db1d9
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/frontend/src/lib/components/OAuthSetting.svelte b/frontend/src/lib/components/OAuthSetting.svelte
@@ -9,7 +9,7 @@
 
 	$: enabled = value != undefined
 
-	let allowed_domains = value?.['allowed_domains'] ?? ''
+	let allowed_domains = value?.['allowed_domains']?.join(', ') ?? ''
 
 	let tenant: string = ''
 	$: name == 'microsoft' && changeTenantId(tenant)
@@ -66,7 +66,8 @@
 				</label>
 			{:else if login}
 				<label class="block pb-2">
-					<span class="text-primary font-semibold text-sm">Allowed domain</span>
+					<span class="text-primary font-semibold text-sm">Allowed domains</span>
+					<span class="text-secondary text-xs">Separate domains with commas</span>
 					<input
 						type="text"
 						placeholder="windmill.dev"
@@ -75,7 +76,7 @@
 							if (allowed_domains == '') {
 								value['allowed_domains'] = undefined
 							} else {
-								value['allowed_domains'] = [allowed_domains]
+								value['allowed_domains'] = allowed_domains.split(',')
 							}
 						}}
 					/>