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

Update FB GRAPH API version to v20.0 #2751

Merged
merged 1 commit into from
May 24, 2024

Conversation

rawdreeg
Copy link
Contributor

@rawdreeg rawdreeg commented May 22, 2024

Changes proposed in this Pull Request:

Update Facebook Marketing API to v20.0

The endpoints we use have not changed, so the application should behave as it used to without any additional changes.

  • Do the changed files pass phpcs checks? Please remove phpcs:ignore comments in changed files and fix any issues, or delete if not practical.

Detailed test instructions:

  1. Use https://github.com/woocommerce/facebook-for-woocommerce/wiki/Smoke-tests for testing instructions
    Make sure that logging is enabled in the plugin. After the tests, inspect the logs for errors.
    The use of v20 API will be clearly visible in the endpoint URL:
Screenshot 2024-05-22 at 14 59 53

Additional information:

I created a related PR to update the api.woocommerce.com/integrations bridge. This will need to be reviewed to ensure that the connection flow also works:

https://github.com/Automattic/woocommerce-connect-server/pull/2487

Changelog entry

Tweak - Bump Marketing API version to v20.0

@github-actions github-actions bot added the changelog: tweak Small change, that isn't actually very important. label May 22, 2024
@rawdreeg rawdreeg self-assigned this May 22, 2024
@rawdreeg rawdreeg requested a review from a team May 22, 2024 13:06
Copy link
Contributor

@ibndawood ibndawood 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, @rawdreeg.

@rawdreeg rawdreeg merged commit 9e35393 into develop May 24, 2024
5 checks passed
@rawdreeg rawdreeg deleted the tweak/upgrade-graph_api-version branch May 24, 2024 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog: tweak Small change, that isn't actually very important.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants