Skip to content
Snippets Groups Projects
Unverified Commit 01309bd1 authored by Brian Mesick's avatar Brian Mesick Committed by GitHub
Browse files

Merge pull request #19164 from edx/bmedx/bulk_cleanup_to_post

Change bulk delete retirements to a POST
parents e876a35c bff1cf3b
No related merge requests found
......@@ -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
......
......@@ -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', ...]
......
......@@ -59,7 +59,7 @@ RETIREMENT_UPDATE = AccountRetirementStatusView.as_view({
})
RETIREMENT_CLEANUP = AccountRetirementStatusView.as_view({
'delete': 'cleanup',
'post': 'cleanup',
})
RETIREMENT_POST = AccountRetirementView.as_view({
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment