Skip to content

Conversation

@seanofahey
Copy link

Thanks for the work to build this SurveyMonkey library for R. I am updating the functions to work with the new SurveyMonkey V3 API and wanted to share the work with you so you could consider merging them back into the main branch. I am working my way through the functions but so far have updated smlogin, surveydetails, surveylist, userdetails and part of getresponses. More to come.

Sean

…removed body JSON now in url, modified class structure line, added new function call to get question_details
…o remove HTML tags from heading responses. updated surveypreview function to match new api data structure
…y as an input instead of using the survey detail to offer consistency across functions
…ls added to allow input of survey or collector ids and bulk responses.
…resh manual pages and NAMESPACE file. Updated function calls in code to explicitly state package::function. Changed internal variable "content" to "parsed_content" to avoid confusion with function "content"
Merge branch 'master' of https://github.com/seanofahey/Rmonkey

# Conflicts:
#	RMonkey Demo.R
… to manage text for "other" responses to multiple choice questions.
Merge branch 'master' of https://github.com/seanofahey/Rmonkey

# Conflicts:
#	man/surveyquestions.Rd
…getresponses call to minimize api queries

Merge branch 'master' of https://github.com/seanofahey/Rmonkey

# Conflicts:
#	R/surveyresponses.r
@sfirke
Copy link

sfirke commented Mar 30, 2017

Hi @seanofahey - are you still working on bringing Rmonkey back to life for SurveyMonkey's current API? Where do things stand? This pull request looks promising.

@sfirke
Copy link

sfirke commented Mar 30, 2017

Looks like a lot of smaller functions are showcased in RMonkey Demo.R and then you've switched the workflow for retrieving responses to be all in surveyresponses()? That function seems to be working. README will need to be rewritten as your update changed how responses are retrieved, i.e., users should use surveyresponses(s[[1]]) instead of the previous getallresponses(s[[1]])

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants