From bff1cf3b74f51ef466870e7297547f52e9653d64 Mon Sep 17 00:00:00 2001 From: bmedx Date: Thu, 25 Oct 2018 15:23:13 -0400 Subject: [PATCH] Change bulk delete retirements to a POST --- .../djangoapps/user_api/accounts/tests/test_retirement_views.py | 2 +- openedx/core/djangoapps/user_api/accounts/views.py | 2 +- openedx/core/djangoapps/user_api/urls.py | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/openedx/core/djangoapps/user_api/accounts/tests/test_retirement_views.py b/openedx/core/djangoapps/user_api/accounts/tests/test_retirement_views.py index a1db84f2d1..bb971fc92f 100644 --- a/openedx/core/djangoapps/user_api/accounts/tests/test_retirement_views.py +++ b/openedx/core/djangoapps/user_api/accounts/tests/test_retirement_views.py @@ -1071,7 +1071,7 @@ class TestAccountRetirementCleanup(RetirementTestCase): if data is None: data = {'usernames': self.usernames} - response = self.client.delete(self.url, json.dumps(data), **self.headers) + response = self.client.post(self.url, json.dumps(data), **self.headers) print(response) self.assertEqual(response.status_code, expected_status) return response diff --git a/openedx/core/djangoapps/user_api/accounts/views.py b/openedx/core/djangoapps/user_api/accounts/views.py index 55645a22da..de003f460a 100644 --- a/openedx/core/djangoapps/user_api/accounts/views.py +++ b/openedx/core/djangoapps/user_api/accounts/views.py @@ -806,7 +806,7 @@ class AccountRetirementStatusView(ViewSet): def cleanup(self, request): """ - DELETE /api/user/v1/accounts/update_retirement_status/ + POST /api/user/v1/accounts/retirement_cleanup/ { 'usernames': ['user1', 'user2', ...] diff --git a/openedx/core/djangoapps/user_api/urls.py b/openedx/core/djangoapps/user_api/urls.py index 3c1658bbcf..4eba47566d 100644 --- a/openedx/core/djangoapps/user_api/urls.py +++ b/openedx/core/djangoapps/user_api/urls.py @@ -59,7 +59,7 @@ RETIREMENT_UPDATE = AccountRetirementStatusView.as_view({ }) RETIREMENT_CLEANUP = AccountRetirementStatusView.as_view({ - 'delete': 'cleanup', + 'post': 'cleanup', }) RETIREMENT_POST = AccountRetirementView.as_view({