-
Notifications
You must be signed in to change notification settings - Fork 25
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 GraphQL support #118
Comments
@GregSutcliffe @maelle @benubah @LucyMcGowan @cvitolo @jennybc @Athanasiamo @olgamie @RickPack @malcolmbarrett @brooklynbagel @hfrick We scheduled a meeting to discuss this on Weds, Nov 17th at 11am PT. If any of you want to join, give a thumbs up and I will send you the invite. |
Its a little late for me in the evening, but keep me posted. I'd like to be involved as I rely on the pkg for the new website build. |
got the invite 👍 |
Update: @schloerke is currently working on rewriting the internal query logic and updating the two most useful functions, |
I can help with testing - let me know when. |
@jromanowska Thank you! @benubah Please let @jromanowska how she can help, when you're ready. |
Just to let everyone know... I am working on rewriting the remaining functions. I have working drafts but will send a PR soon so we can start testing. |
The work (plus testing) is ongoing here: #138 |
Thank you, @benubah! |
Great to see that this package is maintained and that the GraphQL support is being added! When can we expect the updates to be merged into the main branch? |
@frederikho I just merged into master. Please try it out if you can! We would really appreciate feedback. Thanks. |
Meetup will be deprecating the REST API and replacing it with GraphQL. The REST API will be deprecated at the end of 2021, and the GraphQL API is already active. Therefore, for this package to remain usable, we have to replace the guts. The ghql package could be useful.
Here’s the info on migrating from REST to GraphQL: https://www.meetup.com/api/migrating
More info about GraphQL: https://www.meetup.com/api/general
FYI: @GregSutcliffe @maelle @benubah @LucyMcGowan @cvitolo @jennybc @Athanasiamo @olgamie @RickPack @malcolmbarrett @brooklynbagel @hfrick
The text was updated successfully, but these errors were encountered: