Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Brand logo URL is hardcoded and bypasses URL overrides for custom branding #176

Closed
Tracked by #217
arslanashraf7 opened this issue Jul 11, 2023 · 1 comment
Closed
Tracked by #217

Comments

@arslanashraf7
Copy link

arslanashraf7 commented Jul 11, 2023

Description:

While working on an MFE branding/customization and testing the dashboard MFE locally, I noticed that the brand logo wasn't being overridden.

I tried overriding the logo with both possible options:

  1. Added a URL for LOGO_URL in the environment variable through .env.development
  2. Replaced the logo asset/image in the brand repository that I created

Possible Cause

Further digging into this, I noticed that the logo URL in the dashboard MFE's BrandLogo component is hardcoded. I assume this means it would never care for any logo URLs added through environment vars.

Steps to reproduce

  • Setup learner dashboard MFE locally, add a value for LOGO_URL in the .env.development
  • Load the dashboard, You should see the edX logo instead of the one from the manually added URL in step# 1.

Expected behavior

  • The Brand logo is overridable through the URL added in the environment variables
@arbrandes
Copy link
Contributor

The fix is in master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants