Merge pull request #9459 from…

Merge pull request #9459 from edx/tasawer/dev-story/ecom-1017-removing-experimental-logistration-endpoints

Remove experimental logistration endpoints

common/djangoapps/student/views.py

@@ -1021,24 +1021,6 @@ def change_enrollment(request, check_access=True):
         return HttpResponseBadRequest(_("Enrollment action is invalid"))
 
 
-@never_cache
-@ensure_csrf_cookie
-def accounts_login(request):
-    """Deprecated. To be replaced by :class:`student_account.views.login_and_registration_form`."""
-    external_auth_response = external_auth_login(request)
-    if external_auth_response is not None:
-        return external_auth_response
-
-    redirect_to = get_next_url_for_login_page(request)
-    context = {
-        'login_redirect_url': redirect_to,
-        'pipeline_running': 'false',
-        'pipeline_url': auth_pipeline_urls(pipeline.AUTH_ENTRY_LOGIN, redirect_url=redirect_to),
-        'platform_name': settings.PLATFORM_NAME,
-    }
-    return render_to_response('login.html', context)
-
-
 # Need different levels of logging
 @ensure_csrf_cookie
 def login_user(request, error=""):  # pylint: disable=too-many-statements,unused-argument

lms/urls.py

@@ -95,15 +95,12 @@ if settings.FEATURES["ENABLE_COMBINED_LOGIN_REGISTRATION"]:
             {'initial_mode': 'login'}, name="signin_user"),
         url(r'^register$', 'student_account.views.login_and_registration_form',
             {'initial_mode': 'register'}, name="register_user"),
-        url(r'^accounts/login$', 'student_account.views.login_and_registration_form',
-            {'initial_mode': 'login'}, name="accounts_login"),
     )
 else:
     # Serve the old views
     urlpatterns += (
         url(r'^login$', 'student.views.signin_user', name="signin_user"),
         url(r'^register$', 'student.views.register_user', name="register_user"),
-        url(r'^accounts/login$', 'student.views.accounts_login', name="accounts_login"),
     )
 
 if settings.FEATURES.get("ENABLE_CREDIT_API"):