Favicon and siteName not rendering in the title on Tutor

[Danyal-Faheem]

Issue

When we use this MFE on Tutor, I haven't had the opportunity to test it on devstack, the favicon and siteName in the title do not render. Similarly, the title is not translated into the current language, it always shows in English as is defined in the index.html file:

frontend-app-discussions/public/index.html

Line 4 in ac17fd7

<title>Discussions | <%= process.env.SITE_NAME %></title>

Possible Reason

This is likely due to this MFE trying to access the favicon directly in the assets folder as it is being done here:

frontend-app-discussions/src/index.jsx

Line 17 in ac17fd7

import './assets/favicon.ico';

In the other MFEs, they use the react-helmet package to override the head tag, similar to how it is being done in the profile MFE here.

How to reproduce

1. Install any version of tutor
2. Create a superuser using the command: tutor local do createuser --staff --superuser yourusername user@email.com
3. Sign in to tutor with the created user
4. Import the demo course using the command: tutor local do importdemocourse
5. Visit the discussions page for the demo course at this url
6. The favicon and the sitename will not render.

Or you can just visit the sandbox environment, which is running Tutor version 17.0.0. You can visit it at this link