fix: ENABLED_ORG_LOGO definition

fccn · Jul 1, 2024 · e1f358a · e1f358a
1 parent 736a5c7
commit e1f358a
Show file tree

Hide file tree

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

diff --git a/src/__snapshots__/Header.test.jsx.snap b/src/__snapshots__/Header.test.jsx.snap
@@ -93,7 +93,7 @@ exports[`<Header /> renders correctly for anonymous mobile 1`] = `
           height="24px"
           role="img"
           style={
-            {
+            Object {
               "height": "1.5rem",
               "width": "1.5rem",
             }
@@ -163,7 +163,7 @@ exports[`<Header /> renders correctly for anonymous mobile 1`] = `
         <span
           className="avatar overflow-hidden d-inline-flex rounded-circle null"
           style={
-            {
+            Object {
               "height": "1.5rem",
               "width": "1.5rem",
             }
@@ -175,7 +175,7 @@ exports[`<Header /> renders correctly for anonymous mobile 1`] = `
             height="24px"
             role="img"
             style={
-              {
+              Object {
                 "height": "1.5rem",
                 "width": "1.5rem",
               }
@@ -253,7 +253,7 @@ exports[`<Header /> renders correctly for authenticated desktop 1`] = `
             <span
               className="avatar overflow-hidden d-inline-flex rounded-circle mr-2"
               style={
-                {
+                Object {
                   "height": "1.5em",
                   "width": "1.5em",
                 }
@@ -265,7 +265,7 @@ exports[`<Header /> renders correctly for authenticated desktop 1`] = `
                 height="24px"
                 role="img"
                 style={
-                  {
+                  Object {
                     "height": "1.5em",
                     "width": "1.5em",
                   }
@@ -339,7 +339,7 @@ exports[`<Header /> renders correctly for authenticated mobile 1`] = `
           height="24px"
           role="img"
           style={
-            {
+            Object {
               "height": "1.5rem",
               "width": "1.5rem",
             }
@@ -409,7 +409,7 @@ exports[`<Header /> renders correctly for authenticated mobile 1`] = `
         <span
           className="avatar overflow-hidden d-inline-flex rounded-circle null"
           style={
-            {
+            Object {
               "height": "1.5rem",
               "width": "1.5rem",
             }
@@ -421,7 +421,7 @@ exports[`<Header /> renders correctly for authenticated mobile 1`] = `
             height="24px"
             role="img"
             style={
-              {
+              Object {
                 "height": "1.5rem",
                 "width": "1.5rem",
               }

diff --git a/src/learning-header/LearningHeader.jsx b/src/learning-header/LearningHeader.jsx
@@ -25,15 +25,12 @@ LinkedLogo.propTypes = {
   src: PropTypes.string.isRequired,
   alt: PropTypes.string.isRequired,
 };
-
-// this feature flag is not included on the frontend-platform, we have to get it directly from ENV
-const enabledOrgLogo = process.env.ENABLED_ORG_LOGO || false;
-
 const LearningHeader = ({
   courseOrg, courseTitle, intl, showUserDropdown,
 }) => {
   const { authenticatedUser } = useContext(AppContext);
   const [logoOrg, setLogoOrg] = useState(null);
+  const enabledOrgLogo = getConfig().ENABLED_ORG_LOGO || false;
 
   useEffect(() => {
     if (courseOrg) {