Merge pull request #30566 from openedx/ammar/add-source-query-param-in-dsc-url
feat: add source query param in data sharing consent url
This commit is contained in:
@@ -753,6 +753,7 @@ def get_enterprise_consent_url(request, course_id, user=None, return_to=None, en
|
||||
url_params = {
|
||||
'enterprise_customer_uuid': enterprise_customer_uuid_for_request(request),
|
||||
'course_id': course_id,
|
||||
'source': 'lms-courseware',
|
||||
'next': request.build_absolute_uri(return_path),
|
||||
'failure_url': request.build_absolute_uri(
|
||||
reverse('dashboard') + '?' + urlencode(
|
||||
|
||||
@@ -709,6 +709,7 @@ class TestEnterpriseApi(EnterpriseServiceMockMixin, CacheIsolationTestCase):
|
||||
expected_path = request_mock.path if is_return_to_null else '/courses/course-v1:edX+DemoX+Demo_Course/info'
|
||||
expected_url_args = {
|
||||
'course_id': ['course-v1:edX+DemoX+Demo_Course'],
|
||||
'source': ['lms-courseware'],
|
||||
'failure_url': ['http://localhost:8000/dashboard?consent_failed=course-v1%3AedX%2BDemoX%2BDemo_Course'],
|
||||
'enterprise_customer_uuid': ['cf246b88-d5f6-4908-a522-fc307e0b0c59'],
|
||||
'next': [f'http://localhost:8000{expected_path}']
|
||||
|
||||
@@ -25,7 +25,7 @@ django-storages<1.9
|
||||
# The team that owns this package will manually bump this package rather than having it pulled in automatically.
|
||||
# This is to allow them to better control its deployment and to do it in a process that works better
|
||||
# for them.
|
||||
edx-enterprise==3.49.7
|
||||
edx-enterprise==3.49.9
|
||||
|
||||
# oauthlib>3.0.1 causes test failures ( also remove the django-oauth-toolkit constraint when this is fixed )
|
||||
oauthlib==3.0.1
|
||||
|
||||
@@ -467,7 +467,7 @@ edx-drf-extensions==8.0.1
|
||||
# edx-rbac
|
||||
# edx-when
|
||||
# edxval
|
||||
edx-enterprise==3.49.7
|
||||
edx-enterprise==3.49.9
|
||||
# via
|
||||
# -c requirements/edx/../constraints.txt
|
||||
# -r requirements/edx/base.in
|
||||
|
||||
@@ -580,7 +580,7 @@ edx-drf-extensions==8.0.1
|
||||
# edx-rbac
|
||||
# edx-when
|
||||
# edxval
|
||||
edx-enterprise==3.49.7
|
||||
edx-enterprise==3.49.9
|
||||
# via
|
||||
# -c requirements/edx/../constraints.txt
|
||||
# -r requirements/edx/testing.txt
|
||||
|
||||
@@ -563,7 +563,7 @@ edx-drf-extensions==8.0.1
|
||||
# edx-rbac
|
||||
# edx-when
|
||||
# edxval
|
||||
edx-enterprise==3.49.7
|
||||
edx-enterprise==3.49.9
|
||||
# via
|
||||
# -c requirements/edx/../constraints.txt
|
||||
# -r requirements/edx/base.txt
|
||||
|
||||
Reference in New Issue
Block a user