fix: ENABLED_ORG_LOGO definition

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

Hide file tree

Showing 2 changed files with 2 additions and 4 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/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) {