Merge pull request #406 from openedx/KristinAoki/fix-accessibility-pa…

…ge-link-env-variable fix: variable name for the accessibility link
openedx · Mar 8, 2024 · aa13b49 · aa13b49
2 parents 62bf038 + cff78e0
commit aa13b49
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 5 deletions.
diff --git a/.env.development b/.env.development
@@ -12,7 +12,7 @@ TERMS_OF_SERVICE_URL=null
 PRIVACY_POLICY_URL=null
 SUPPORT_EMAIL=null
 STUDIO_BASE_URL=http://localhost:18010
-SHOW_ACCESSIBILITY_PAGE=false
+ENABLE_ACCESSIBILITY_PAGE=false
 ORDER_HISTORY_URL=localhost:1996/orders
 REFRESH_ACCESS_TOKEN_ENDPOINT=http://localhost:18000/login_refresh
 SEGMENT_KEY=null

diff --git a/src/components/studio-footer/StudioFooter.jsx b/src/components/studio-footer/StudioFooter.jsx
@@ -22,7 +22,7 @@ ensureConfig([
   'SUPPORT_EMAIL',
   'SITE_NAME',
   'STUDIO_BASE_URL',
-  'SHOW_ACCESSIBILITY_PAGE',
+  'ENABLE_ACCESSIBILITY_PAGE',
 ], 'Studio Footer component');
 
 const StudioFooter = ({
@@ -105,7 +105,7 @@ const StudioFooter = ({
               {intl.formatMessage(messages.privacyPolicyLinkLabel)}
             </Hyperlink>
           )}
-          {config.SHOW_ACCESSIBILITY_PAGE === 'true' && (
+          {config.ENABLE_ACCESSIBILITY_PAGE === 'true' && (
             <Hyperlink
               destination={`${config.STUDIO_BASE_URL}/accessibility`}
               data-testid="accessibilityRequest"

diff --git a/src/components/studio-footer/StudioFooter.test.jsx b/src/components/studio-footer/StudioFooter.test.jsx
@@ -15,7 +15,7 @@ const config = {
   SUPPORT_EMAIL: process.env.SUPPORT_EMAIL,
   SITE_NAME: process.env.SITE_NAME,
   STUDIO_BASE_URL: process.env.STUDIO_BASE_URL,
-  SHOW_ACCESSIBILITY_PAGE: process.env.SHOW_ACCESSIBILITY_PAGE,
+  ENABLE_ACCESSIBILITY_PAGE: process.env.ENABLE_ACCESSIBILITY_PAGE,
 };
 
 let currentConfig = config;
@@ -111,7 +111,7 @@ describe('Footer', () => {
       expect(screen.queryByTestId('accessibilityRequest')).toBeNull();
     });
     it('should show accessibilty request link', () => {
-      render(<Component updateVariable={['SHOW_ACCESSIBILITY_PAGE', 'true']} />);
+      render(<Component updateVariable={['ENABLE_ACCESSIBILITY_PAGE', 'true']} />);
       expect(screen.getByText('LMS')).toBeVisible();
       expect(screen.queryByTestId('termsOfService')).toBeNull();
       expect(screen.queryByTestId('privacyPolicy')).toBeNull();

diff --git a/src/setupTest.js b/src/setupTest.js
@@ -19,3 +19,4 @@ process.env.LOGO_URL = 'https://edx-cdn.org/v3/default/logo.svg';
 process.env.LOGO_TRADEMARK_URL = 'https://edx-cdn.org/v3/default/logo-trademark.svg';
 process.env.LOGO_WHITE_URL = 'https://edx-cdn.org/v3/default/logo-white.svg';
 process.env.FAVICON_URL = 'https://edx-cdn.org/v3/default/favicon.ico';
+process.env.ENABLE_ACCESSIBILITY_PAGE = 'false';