fix: browser header showing null

[attiyaIshaque]

Description

When accessing Authn MFE for the first time it shows “Auth | null” in the browser header. In this PR we fix If process.env.SITE_NAME is not available it will show “Authn” otherwise show “Authn | ”

Backport PR: #1170

JIRA

VAN-1839

How Has This Been Tested?

Please describe in detail how you tested your changes.

Screenshots/sandbox (optional):

Before	After

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.23%. Comparing base (9b04614) to head (4758cd4).

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1168   +/-   ##
=======================================
  Coverage   84.23%   84.23%           
=======================================
  Files         127      127           
  Lines        2392     2392           
  Branches      678      678           
=======================================
  Hits         2015     2015           
  Misses        357      357           
  Partials       20       20           

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

[syedsajjadkazmii]

Nit: its totally up to you if you think its more readable

<title><%= 'Authn' + (process.env.SITE_NAME ? ' | ' + process.env.SITE_NAME : '') %></title>

[regisb]

Can I suggest you also update the MFE template itself? https://github.com/openedx/frontend-template-application/blob/master/public/index.html this is the reference repo that is used to generate all MFE repos.

[hinakhadim]

Can I suggest you also update the MFE template itself? https://github.com/openedx/frontend-template-application/blob/master/public/index.html this is the reference repo that is used to generate all MFE repos.

Updated in frontend-template-application