fix: get brand logo file path from env (#205)

[shahbaz-shabbir05]

This PR makes changes so that the LOGO_URL can be used from the config in the Quince branch. These changes have already been made and merged in the master branch.

The PR for the master branch is linked here: #205 & #238

Additional context: #178

[openedx-webhooks]

Thanks for the pull request, @shahbaz-shabbir05! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

• supporting documentation
• Open edX discussion forum threads
• timeline information ("this must be merged by XX date", and why that is)
• partner information ("this is a course on edx.org")
• any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (open-release/quince.master@2cf6e5a). Click here to learn what that means.

Additional details and impacted files

@@                      Coverage Diff                      @@
##             open-release/quince.master     #252   +/-   ##
=============================================================
  Coverage                              ?   96.46%           
=============================================================
  Files                                 ?      195           
  Lines                                 ?     1839           
  Branches                              ?      322           
=============================================================
  Hits                                  ?     1774           
  Misses                                ?       60           
  Partials                              ?        5           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[arslanashraf7]

@arbrandes or @mariajgrimaldi Hi, This is the backport of #205 to fix the hardcoded brand logo URL in this Quince release. Could any of you review or comment?

[arbrandes]

@shahbaz-shabbir05, FYI, I'm aiming to pull in everything that's on master into Quince via #243.

[shahbaz-shabbir05]

@shahbaz-shabbir05, FYI, I'm aiming to pull in everything that's on master into Quince via #243.

@arbrandes Thank you for the update. I was wondering if there's a specific timeline for when the Quince branch will be synchronized with the master branch, just for my information.

Also, you're planning to add everything from the master into Quince, should I close this PR then?

[brian-smith-tcril]

I believe this should be handled by backporting #232 so we can use runtime config instead.

[shahbaz-shabbir05]

I believe this should be handled by backporting #232 so we can use runtime config instead.

@brian-smith-tcril Appreciate the update. I'll proceed to close this PR now.

[openedx-webhooks]

@shahbaz-shabbir05 Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future.