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

Updated Graph API version #91

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Updated Graph API version #91

wants to merge 1 commit into from

Conversation

magnusburton
Copy link

Thought we would update the Graph API version from v2.6 to v2.11 😄

@mraaroncruz
Copy link
Collaborator

I don't know what the repercussions of this would be, that is why i haven't merged this.
@Charca any insights here?

@Charca
Copy link
Owner

Charca commented Dec 19, 2017

I didn't see any relevant updates in v2.11 here https://developers.facebook.com/docs/graph-api/changelog

@magnusburton what are the features for the Messenger API that are available in the new version?

And in case there are breaking changes, I think the right way to do this would be to make the version a param that can be configured when creating the BootBot instance (and default to v2.6). Thoughts?

@magnusburton
Copy link
Author

@Charca I believe the new Broadcast API is running on v2.11+ but since you haven't implemented support for it it might be unnecessary at this point in time.

I recently upgraded my local copy of Bootbot to v2.11 and have yet to experience breaking changes. Including a parameter for the version would be the best option in this case.

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.

3 participants