feat: add LOGISTRATION_MINIMAL_HEADER env variable #75
Conversation
waheedahmed
force-pushed
the
waheed/hide-header-links
branch
2 times, most recently
from
d6fde3c to
5d2b9e4
Compare
waheedahmed
changed the title
|
@davidjoy Do we also need to do this in @edx/frontend-component-header-edx or the logo is configurable in this component so that we can change the logo to edX logo when we go live? I know there is a similar env variable ( |
|
The @edx/frontend-component-header-edx is what actually gets used on edx.org - this repository does not get used in production. I'd suggest adding this variable in that repo and leaving this one as-is. |
|
@davidjoy I believe we need this in both to keep the user experience same for the edX and OpenedX community. I'll make a PR for @edx/frontend-component-header-edx as well. |
The LOGISTRATION_MINIMAL_HEADER environment variable removes the main menu links from the header. It also removes login/register links from the menu if user is unauthenticated.
waheedahmed
force-pushed
the
waheed/hide-header-links
branch
from
5d2b9e4 to
2f2c2f7
Compare
|
Created PR for edX specific header component as well edx/frontend-component-header-edx#105. |
The
LOGISTRATION_MINIMAL_HEADERenvironment variable removes the main menu links from the header. It also removes login/register links from the menu if user is unauthenticated.