openedx / openedx-demo-course

A demonstration course that can be imported into an Open edX instance
GNU Affero General Public License v3.0
44 stars 68 forks source link

fix: change the default image title (BACKPORT) #44

Closed DmytroAlipov closed 1 year ago

DmytroAlipov commented 1 year ago

This is a backport from the master branch. PR to master

The platform's default image title for staff is placeholder-faculty.png. However, in the default course, the image link still has title pl-faculty.png. Because of this, the default image is not displayed on the About course page. It is like this now: screen_1

For any created course where an image has not been set for the staff: screen_2

I manually tried to input the existing title of the image. Here's the result: screen_3

openedx-webhooks commented 1 year ago

Thanks for the pull request, @DmytroAlipov! 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:

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.

itsjeyd commented 1 year ago

Thanks @DmytroAlipov! It looks like this is ready for review (but let me know if you were planning to work on it more).

@sarina Since you reviewed the main PR, would you mind having a quick look at this Palm backport as well?

openedx-webhooks commented 1 year ago

@DmytroAlipov 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.