-
Notifications
You must be signed in to change notification settings - Fork 0
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
API List 2 DF: Part 1 - Google Analytics #16
Conversation
@howardbaek This is ready for your review. And if it runs okay for you then the output data for ITCR account for the Hopefully writing it flexibly enough that the other data sources will be able to be added to it later. |
API List 2 DF: Part 2 - GitHub #16
@howardbaek can you test if the following works for you on this branch?
|
I ran this line of code and got a list of length 3: |
Perfect. That is what we expect. |
Purpose/implementation Section
What changes are being implemented in this Pull Request?
#13 First stop on the road for data to be turned into nice data frames. There will be a series of these types of PRs where I'm trying to make custom functions that turn our API lists into nice dataframes for easy use!
What was your approach?
Made custom functions that pull out the relevant data and put it into a nice data frame format!
Now
all_ga_metrics()
function that I developed in #11 has an argument that by default is set to "dataframe" which means we attempt to convert the gnarly API list into a nice data.frame. However, if someone wanted the original result they could just change this to "raw".