Skip to content
Snippets Groups Projects
Unverified Commit fb73bcb2 authored by Feanil Patel's avatar Feanil Patel Committed by GitHub
Browse files

Merge pull request #22478 from edx/diana/handle-branding-string

Redirect to the view names directly.
parents 5d095b3a 6e7d54e8
Branches
Tags
No related merge requests found
......@@ -43,7 +43,7 @@ def index(request):
if configuration_helpers.get_value(
'ALWAYS_REDIRECT_HOMEPAGE_TO_DASHBOARD_FOR_AUTHENTICATED_USER',
settings.FEATURES.get('ALWAYS_REDIRECT_HOMEPAGE_TO_DASHBOARD_FOR_AUTHENTICATED_USER', True)):
return redirect(reverse('dashboard'))
return redirect('dashboard')
enable_mktg_site = configuration_helpers.get_value(
'ENABLE_MKTG_SITE',
......@@ -62,7 +62,7 @@ def index(request):
# keep specialized logic for Edge until we can migrate over Edge to fully use
# configuration.
if domain and 'edge.edx.org' in domain:
return redirect(reverse("signin_user"))
return redirect("signin_user")
# we do not expect this case to be reached in cases where
# marketing and edge are enabled
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment