Skip to content
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

Add functions for meetup pro accounts #92

Merged
merged 24 commits into from
Jan 12, 2021
Merged

Add functions for meetup pro accounts #92

merged 24 commits into from
Jan 12, 2021

Conversation

drmowinckels
Copy link
Member

@drmowinckels drmowinckels commented Jan 7, 2021

Fix #89
Fix #94

This PR replaces #61, whose PR'ing repo has been deleted.

The pro URLs are returning 401, for unknown reasons.
But the meetup API console is also returning 401's when I try it there.

I'm wondering if getting information on the pro groups this way is restricted to pro users somehow, though I cant see that documented.

Copy link
Contributor

@maelle maelle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!
I suppose that this can't be tested yet as we get the 401 😬

R/get_pro.R Outdated Show resolved Hide resolved
R/get_pro.R Outdated Show resolved Hide resolved
R/get_pro.R Outdated Show resolved Hide resolved
R/get_pro.R Outdated Show resolved Hide resolved
R/internals.R Outdated Show resolved Hide resolved
@maelle maelle merged commit a8a8009 into master Jan 12, 2021
@maelle maelle deleted the pro branch January 12, 2021 10:07
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.

Make verbose controllable by an option If there are no events, return empty tibble instead of an error
2 participants