REDCap now supports delete through API

[damonms]

Are there any plans to upgrade PyCap to include delete for records? If not, I will take a stab at it. It shouldn't take long.

Cheers from Steve and the VUIIS folks

[sburns]

Hi Steve!

Currently no plans but I'd of course welcome a PR. I think they finally revoked my vandy credentials from REDCap, so if you submit a PR it will likely fail CI :(

[damonms]

OK. I have a fix already working. I'll get a PR formulated and submit for approval. Cheers, Steve

…

________________________________ From: Scott Burns [notifications@github.com] Sent: Monday, July 24, 2017 10:04 AM To: redcap-tools/PyCap Cc: Damon, Stephen M; Author Subject: Re: [redcap-tools/PyCap] REDCap now supports delete through API (#76) Hi Steve! Currently no plans but I'd of course welcome a PR. I think they finally revoked my vandy credentials from REDCap, so if you submit a PR it will likely fail CI :( — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub<#76 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AGqurcrpKPxC7rPc4_xawhCXDCd2R7ONks5sRLKLgaJpZM4OhNdo>.

[remifrazierucsf]

Steve – keep us posted; I’ve been extending some classes as well and maybe we could merge. From: Stephen Damon [mailto:notifications@github.com] Sent: Monday, July 24, 2017 8:13 AM To: redcap-tools/PyCap <PyCap@noreply.github.com> Cc: Subscribed <subscribed@noreply.github.com> Subject: Re: [redcap-tools/PyCap] REDCap now supports delete through API (#76) OK. I have a fix already working. I'll get a PR formulated and submit for approval. Cheers, Steve

…

________________________________ From: Scott Burns [notifications@github.com] Sent: Monday, July 24, 2017 10:04 AM To: redcap-tools/PyCap Cc: Damon, Stephen M; Author Subject: Re: [redcap-tools/PyCap] REDCap now supports delete through API (#76) Hi Steve! Currently no plans but I'd of course welcome a PR. I think they finally revoked my vandy credentials from REDCap, so if you submit a PR it will likely fail CI :( — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub<#76 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AGqurcrpKPxC7rPc4_xawhCXDCd2R7ONks5sRLKLgaJpZM4OhNdo>. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub<#76 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AbyWnBX0zbpeXXqkoUs-876mE2rLzB8Iks5sRLSWgaJpZM4OhNdo>.

[bobromeo]

The title of this issue is not yet OK :), the Pull Request is not yet handled.
Looked into files of the branch, 'delete_record' only deletes 1 record, so the README.rst should be changed to singular.
-OR-
Better still: the .py could be changed to allow multiple records to be deleted, because the API allows for records[0], records[1] etc.
I am a noob at GitHub and Python is not my forté, but here's my 3cts solution for deleting records.
This works, but i am not yet using records[1] etc, have to dig in a bit deeper :)
Also: a nice 'delete_records' should iterate over chunks of 1000 or so ...

dataRep = {
    'token': api_key,
    'content': 'report',
    'format': 'csv',
    'report_id': '16',
    'rawOrLabel': 'raw',
    'rawOrLabelHeaders': 'raw',
    'exportCheckboxLabel': 'false',
    'returnFormat': 'csv'
}
dataDel = {
    'token': api_key,
    'action': 'delete',
    'content': 'record',
    'records[0]': 'xxx'
}
rr = requests.post(api_url, data = dataRep, verify=False)
for rrl in rr.iter_lines(chunk_size=128,decode_unicode=True):
    if(rrl != 'record_id'):
        dataDel["records[0]"] = rrl.strip('\"')
        rd = requests.post(api_url, data = dataDel, verify=False)
rr = requests.post(api_url, data = dataRep, verify=False)
print(rr.text)

[pwildenhain]

See my comment in the PR for how to easily delete multiple records through the API: #77 (comment)
TL:DR Change records[0] to records[] in the body of the POST request to pass a list of records to be deleted