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

Fix native Facebook graph query to retrieve engagement counts #98

Merged
merged 2 commits into from
May 18, 2020

Conversation

NormanHoehne
Copy link
Contributor

Description

I have added a new argument 'fields' to the Facebook api query to make it possible to retrieve count statistics again. Since the api response has changed, I have also updated the lines that assign the counts.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (modification of the currently available functionality)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project (WordPress coding standards, etc).
  • My code has appropriate phpdoc comments with a description, @since, @params and @return.
  • My code is tested for both new installs and for users that are upgrading from older versions.

@billerickson
Copy link
Collaborator

Thank you!

@billerickson billerickson merged commit b230d16 into jaredatch:develop May 18, 2020
billerickson added a commit that referenced this pull request May 18, 2020
A Facebook Access Token is now required to retrieve native FB counts, see #96 and #98
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.

2 participants