fix: rename ENABLED_ORG_LOGO setting

Rename the setting to be in the present tense, as a way to make it clearer. The setting is now called `ENABLE_ORG_LOGO`.
fccn · Aug 20, 2024 · 9d8b7a9 · 9d8b7a9
1 parent a8789fe
commit 9d8b7a9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/Header.jsx b/src/Header.jsx
@@ -28,7 +28,7 @@ ensureConfig([
 subscribe(APP_CONFIG_INITIALIZED, () => {
   mergeConfig({
     AUTHN_MINIMAL_HEADER: !!process.env.AUTHN_MINIMAL_HEADER,
-    ENABLED_ORG_LOGO: !!process.env.ENABLED_ORG_LOGO,
+    ENABLE_ORG_LOGO: !!process.env.ENABLE_ORG_LOGO,
   }, 'Header additional config');
 });
 

diff --git a/src/learning-header/LearningHeader.jsx b/src/learning-header/LearningHeader.jsx
@@ -30,7 +30,7 @@ const LearningHeader = ({
 }) => {
   const { authenticatedUser } = useContext(AppContext);
   const [logoOrg, setLogoOrg] = useState(null);
-  const enabledOrgLogo = getConfig().ENABLED_ORG_LOGO || false;
+  const enableOrgLogo = getConfig().ENABLE_ORG_LOGO || false;
 
   useEffect(() => {
     if (courseOrg) {
@@ -53,8 +53,8 @@ const LearningHeader = ({
       <div className="container-xl py-2 d-flex align-items-center">
         {headerLogo}
         <div className="d-none d-md-block flex-grow-1 course-title-lockup">
-          <div className={`d-md-flex ${enabledOrgLogo && 'align-items-center justify-content-center'} w-100`}>
-            {enabledOrgLogo ? (
+          <div className={`d-md-flex ${enableOrgLogo && 'align-items-center justify-content-center'} w-100`}>
+            {enableOrgLogo ? (
               (courseOrg && logoOrg)
               && <img src={logoOrg} alt={`${courseOrg} logo`} style={{ maxHeight: '45px' }} />
             ) : null}