using jsonlite to automatically build payloads #155
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Suggested change--still needs tests written, but thought I'd go ahead and put it in the pipe for review. So far seems to work fine.
Uses
toJSON
from packagejsonlite
for automated construction of payloads, eliminating the massive manual build using a bunch ofifelse
s andpaste
.qualtRics
has imported functions fromjsonlite
for quite a while now, so this doesn't add any new dependencies.purrr::discard
drops any null parameters when building, as qualtrics API doesn't like actual jsonnulls
or empty sets.Retains original technique for formatting
start_date
andend_date
to fit ISO 8601 as expected (https://api.qualtrics.com/docs/dates-and-times). Looking ahead, though, could be easier now to set up more flexible handling of start and end times, useful now that since-last-response request functionality is gone, see last_response still fetches all responses #153.