New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New feature #12563 remote control: process files on add_response() #793

Merged
merged 1 commit into from Aug 15, 2017

Conversation

Projects
None yet
3 participants
@pardo-bsso

pardo-bsso commented Aug 14, 2017

Dev: When calling add_response(), if the response data has fields for file
attachments, move those files to the final upload folder if they are not there.
This assumes that the files are already present on the temporary upload folder.

@olleharstedt

This comment has been minimized.

Show comment
Hide comment
@olleharstedt

olleharstedt Aug 14, 2017

Contributor

Looks good to me. Anyone else wants to comment? @Shnoulle @LouisGac

Contributor

olleharstedt commented Aug 14, 2017

Looks good to me. Anyone else wants to comment? @Shnoulle @LouisGac

New feature #12563 remote control: process files on add_response()
Dev: When calling add_response(), if the response data has fields for file
attachments, move those files to the final upload folder if they are not there.
This assumes that the files are already present on the temporary upload folder.

@pardo-bsso pardo-bsso changed the base branch from master to develop Aug 14, 2017

@LouisGac LouisGac merged commit 66c1871 into LimeSurvey:develop Aug 15, 2017

1 check passed

review/gitmate/pr This PR has no issues. :)
Details

@pardo-bsso pardo-bsso deleted the Cambalab:remotecontrol-process-files-in-add_response branch Aug 16, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment