From 7e3966fcca84de1770a57fbe72133692a329081b Mon Sep 17 00:00:00 2001
From: Asad Ali <asad.ali@arbisoft.com>
Date: Thu, 25 Jan 2024 17:23:00 +0500
Subject: [PATCH] refactor: more updates

---
 .env.development                              |  1 -
 README.rst                                    |  1 +
 package.json                                  |  2 +-
 src/components/Footer.jsx                     |  2 +-
 .../__snapshots__/Footer.test.jsx.snap        | 24 +++++++++----------
 src/setupTest.js                              | 11 ++++-----
 6 files changed, 20 insertions(+), 21 deletions(-)

diff --git a/.env.development b/.env.development
index 1600852456..2627b28978 100644
--- a/.env.development
+++ b/.env.development
@@ -28,6 +28,5 @@ CONTACT_URL=
 SUPPORT_CENTER_URL=
 SUPPORT_CENTER_TEXT=
 TRADEMARK_TEXT=
-SITE_URL=
 LOGO_ALT_TEXT=
 SHOW_LOGO=
diff --git a/README.rst b/README.rst
index 4a13dfd8e2..82b8b46569 100644
--- a/README.rst
+++ b/README.rst
@@ -184,6 +184,7 @@ Reporting Security Issues
 =========================
 
 Please do not report security issues in public. Please email security@openedx.org.
+
 .. |Build Status| image:: https://api.travis-ci.com/edx/frontend-component-footer.svg?branch=master
    :target: https://travis-ci.com/edx/frontend-component-footer
 .. |Codecov| image:: https://img.shields.io/codecov/c/github/edx/frontend-component-footer
diff --git a/package.json b/package.json
index ddb9f543b3..934bd10b05 100644
--- a/package.json
+++ b/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "@edx/frontend-component-footer-mitol",
+  "name": "@edx/frontend-component-footer",
   "version": "1.0.0-semantically-released",
   "description": "Footer component for use when building Open edX frontend applications",
   "main": "dist/index.js",
diff --git a/src/components/Footer.jsx b/src/components/Footer.jsx
index aa8dfb65f8..33428ea49c 100644
--- a/src/components/Footer.jsx
+++ b/src/components/Footer.jsx
@@ -79,7 +79,7 @@ class SiteFooter extends React.Component {
                 {this.renderLinkIfExists(process.env.TERMS_OF_SERVICE_URL, 'Terms of Service')}
                 {this.renderLinkIfExists(process.env.PRIVACY_POLICY_URL, 'Privacy Policy')}
                 {this.renderLinkIfExists(process.env.HONOR_CODE_URL, 'Honor Code')}
-                {this.renderLinkIfExists(process.env.Contact, 'Contact')}
+                {this.renderLinkIfExists(process.env.CONTACT_URL, 'Contact Us')}
                 {this.renderLinkIfExists(process.env.ACCESSIBILITY_URL, 'Accessibility')}
                 {this.renderLinkIfExists(process.env.SUPPORT_CENTER_URL, process.env.SUPPORT_CENTER_TEXT || 'FAQ & Help')}
               </ul>
diff --git a/src/components/__snapshots__/Footer.test.jsx.snap b/src/components/__snapshots__/Footer.test.jsx.snap
index df8fe41743..d0a5ae5d7d 100644
--- a/src/components/__snapshots__/Footer.test.jsx.snap
+++ b/src/components/__snapshots__/Footer.test.jsx.snap
@@ -44,7 +44,7 @@ exports[`<Footer /> renders correctly renders with a language selector 1`] = `
           </li>
           <li>
             <a
-              href="http://localhost:18000/tos_and_honor"
+              href="http://localhost:18000/terms-of-service"
             >
               Terms of Service
             </a>
@@ -58,21 +58,21 @@ exports[`<Footer /> renders correctly renders with a language selector 1`] = `
           </li>
           <li>
             <a
-              href="http://localhost:18000/tos_and_honor"
+              href="http://localhost:18000/honor"
             >
               Honor Code
             </a>
           </li>
           <li>
             <a
-              href="https://accessibility.mit.edu/"
+              href="https://www.edx.org/accessibility"
             >
               Accessibility
             </a>
           </li>
           <li>
             <a
-              href="http://localhost:18000/about"
+              href="http://localhost:18000/contact"
             >
               SUPPORT CENTER
             </a>
@@ -170,7 +170,7 @@ exports[`<Footer /> renders correctly renders without a language selector 1`] =
           </li>
           <li>
             <a
-              href="http://localhost:18000/tos_and_honor"
+              href="http://localhost:18000/terms-of-service"
             >
               Terms of Service
             </a>
@@ -184,21 +184,21 @@ exports[`<Footer /> renders correctly renders without a language selector 1`] =
           </li>
           <li>
             <a
-              href="http://localhost:18000/tos_and_honor"
+              href="http://localhost:18000/honor"
             >
               Honor Code
             </a>
           </li>
           <li>
             <a
-              href="https://accessibility.mit.edu/"
+              href="https://www.edx.org/accessibility"
             >
               Accessibility
             </a>
           </li>
           <li>
             <a
-              href="http://localhost:18000/about"
+              href="http://localhost:18000/contact"
             >
               SUPPORT CENTER
             </a>
@@ -257,7 +257,7 @@ exports[`<Footer /> renders correctly renders without a language selector in es
           </li>
           <li>
             <a
-              href="http://localhost:18000/tos_and_honor"
+              href="http://localhost:18000/terms-of-service"
             >
               Terms of Service
             </a>
@@ -271,21 +271,21 @@ exports[`<Footer /> renders correctly renders without a language selector in es
           </li>
           <li>
             <a
-              href="http://localhost:18000/tos_and_honor"
+              href="http://localhost:18000/honor"
             >
               Honor Code
             </a>
           </li>
           <li>
             <a
-              href="https://accessibility.mit.edu/"
+              href="https://www.edx.org/accessibility"
             >
               Accessibility
             </a>
           </li>
           <li>
             <a
-              href="http://localhost:18000/about"
+              href="http://localhost:18000/contact"
             >
               SUPPORT CENTER
             </a>
diff --git a/src/setupTest.js b/src/setupTest.js
index f3913f6ed4..d83743e5b8 100644
--- a/src/setupTest.js
+++ b/src/setupTest.js
@@ -21,14 +21,13 @@ 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.ABOUT_US_URL = 'http://localhost:18000/about';
 process.env.PRIVACY_POLICY_URL = 'http://localhost:18000/privacy';
-process.env.HONOR_CODE_URL = 'http://localhost:18000/tos_and_honor';
-process.env.TERMS_OF_SERVICE_URL = 'http://localhost:18000/tos_and_honor';
-process.env.CONTACT_URL = 'http://localhost:18000/about';
-process.env.SUPPORT_CENTER_URL = 'http://localhost:18000/about';
+process.env.HONOR_CODE_URL = 'http://localhost:18000/honor';
+process.env.TERMS_OF_SERVICE_URL = 'http://localhost:18000/terms-of-service';
+process.env.CONTACT_URL = 'http://localhost:18000/contact';
+process.env.SUPPORT_CENTER_URL = 'http://localhost:18000/contact';
 process.env.SUPPORT_CENTER_TEXT = 'SUPPORT CENTER';
 process.env.TRADEMARK_TEXT = 'Trade Mark text';
-process.env.SITE_URL = 'http://localhost:18000/';
 process.env.LOGO_ALT_TEXT = 'alt text';
 process.env.SHOW_LOGO = true;
 process.env.SUPPORT_EMAIL = 'webmaster@email.com';
-process.env.ACCESSIBILITY_URL = 'https://accessibility.mit.edu/';
+process.env.ACCESSIBILITY_URL = 'https://www.edx.org/accessibility';