daranzolin / rcanvas

R Client for Canvas LMS API
Other
90 stars 43 forks source link

rcanvas:::process_response returns list not data.frame #54

Open schmettow opened 3 years ago

schmettow commented 3 years ago

Hi,

thanks for this amazing work! I just started using rcanvas (v0.0.0.9001), but some get_x functions not work. For example, get_course_items fails with:

Error in UseMethod("mutate") : no applicable method for 'mutate' applied to an object of class "list"

I digged a little into it and to me it seems that the internal function rcanvas:::process_response causes the problem, as it returns a list of (one) data frame. For get_course_items it works, when I change the last line in the function code to be:

rcanvas:::process_response(url, args)[[1]] %>% dplyr::mutate(course_id = course_id)

But, probably, process_response needs a fix.

CU, Martin.

korandabit commented 3 years ago

FWIW I get this error too. it seems process_response was edited in the last commit to address a different issue, and returns a list if it cannot return a df. (But that's the limit of my git/R skills understanding.)