Commit 7179fac6 by tasawernawaz

Merge pull request #9575 from edx/tasawer/story/xcom-285-removing-deprecated-view

removing deprecated view VerifyLaterView
parents c2197608 01ca1f9b
......@@ -326,14 +326,6 @@ class TestPayAndVerifyView(UrlResetMixin, ModuleStoreTestCase, XssTestMixin):
response = self._get_page(page_name, course.id, expected_status_code=302)
self._assert_redirects_to_upgrade(response, course.id)
def test_verify_later(self):
""" The deprecated verify-later page should redirect to the verification start page. """
course = self._create_course("verified")
course_key = course.id
self._enroll(course_key, "verified")
response = self._get_page("verify_student_verify_later", course_key, expected_status_code=301)
self._assert_redirects_to_verify_start(response, course_key, 301)
def test_payment_confirmation(self):
course = self._create_course("verified")
self._enroll(course.id, "verified")
......
......@@ -52,17 +52,6 @@ urlpatterns = patterns(
}
),
# The user has paid and still needs to verify,
# but the user is NOT arriving directly from the payment flow.
# This is equivalent to starting a new flow
# with the payment steps and requirements hidden
# (since the user already paid).
url(
r'^verify-later/{course}/$'.format(course=settings.COURSE_ID_PATTERN),
views.VerifyLaterView.as_view(),
name="verify_student_verify_later"
),
# The user is returning to the flow after paying.
# This usually occurs after a redirect from the shopping cart
# once the order has been fulfilled.
......
......@@ -1473,11 +1473,3 @@ class InCourseReverifyView(View):
params = urllib.urlencode({"checkpoint": checkpoint})
full_url = u"{base}?{params}".format(base=base_url, params=params)
return redirect(full_url)
class VerifyLaterView(RedirectView):
""" This view has been deprecated and should redirect to the unified verification flow. """
permanent = True
def get_redirect_url(self, course_id, **kwargs): # pylint: disable=unused-argument
return reverse('verify_student_verify_now', kwargs={'course_id': unicode(course_id)})
Markdown is supported
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