[ALS-532] Move TOS flag from branding to env variables (#357)

hms-dbmi · Jan 31, 2025 · 126a7ed · 126a7ed
1 parent b2f47ac
commit 126a7ed
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 5 deletions.
diff --git a/src/lib/assets/configuration.json b/src/lib/assets/configuration.json
@@ -37,7 +37,6 @@
     }
   ],
   "footer": {
-    "showTerms": true,
     "showSitemap": true,
     "excludeSitemapOn": ["/explorer", "/discover"],
     "links": [

diff --git a/src/lib/components/Footer.svelte b/src/lib/components/Footer.svelte
@@ -3,7 +3,7 @@
   const modalStore = getModalStore();
 
   import { page } from '$app/stores';
-  import { branding } from '$lib/configuration';
+  import { branding, features } from '$lib/configuration';
   import { user } from '$lib/stores/User';
 
   import TermsModal from '$lib/components/modals/TermsModal.svelte';
@@ -49,8 +49,10 @@
 {/if}
 <footer id="main-footer" class="flex relative">
   <ul>
-    {#if branding?.footer?.showTerms}
-      <li><button class="hover:underline" on:click={openTermsModal}>Terms of Service</button></li>
+    {#if features.termsOfService}
+      <li>
+        <button class="hover:underline text-xs" on:click={openTermsModal}>Terms of Service</button>
+      </li>
     {/if}
     {#each branding?.footer?.links as link}
       <li>

diff --git a/src/lib/configuration.ts b/src/lib/configuration.ts
@@ -141,6 +141,7 @@ export const features: Indexable = {
   dashboard: import.meta.env?.VITE_DASHBOARD === 'true',
   dashboardDrawer: import.meta.env?.VITE_DASHBOARD_DRAWER === 'true',
   confirmDownload: import.meta.env?.VITE_CONFIRM_DOWNLOAD === 'true',
+  termsOfService: import.meta.env?.VITE_ENABLE_TOS === 'true',
 };
 
 export const settings: Indexable = {

diff --git a/src/lib/types.ts b/src/lib/types.ts
@@ -21,7 +21,6 @@ export interface SiteMapConfig {
 
 export interface FooterConfig {
   showSitemap: boolean;
-  showTerms?: boolean;
   excludeSitemapOn: string[];
   links: Array<{
     title: string;