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

Multichannel Marketing Integration - Basic details and methods to report feed status #705

Merged

Conversation

nima-karimi
Copy link
Collaborator

Changes proposed in this Pull Request:

Closes #697.

This PR implements an integration with the new WooCommerce Multichannel Marketing dashboard. I have also done a number of refactoring changes to make the code more reusable. Some minor code style updates are also included in this PR.

Screenshots:

image

Detailed test instructions:

To test this PR it's best if you use the latest development version of WooCommerce (trunk).

  1. Go to WooCommerce Settings > Advanced > Features and turn on the multichannel marketing feature.
  2. Go to the Marketing > Overview page and confirm Pinterest for WooCommerce appears in the "Installed marketing extensions" section
  3. Ensure you have NOT set up the extension or completed onboarding. If you have, reset the extension data.
  4. Confirm that a link to "Finish setup" is displayed next to each extension name if it has not been set up and links to the onboarding/setup process.
  5. Once you complete setup/onboarding, confirm that the "Finish setup" link is replaced with a "Manage" link and points to the extension's settings page.
  6. Confirm that the product sync / feed status is correct and it changes depending on the statuses displayed on the Products Catalog page of the extension
  7. Confirm that the number of errors displayed is consistent with the number of errors in the Products Catalog page

For a compatibility check:

  1. Use a WooCommerce version older than v7.4 (use 7.3, for example) - This means that the MCM feature classes will not be included in the WooCommerce code
  2. Reload the marketing page and confirm that there are no errors loading the page

Additional details:

Changelog entry

@nima-karimi nima-karimi added the changelog: none Skip changelog entry for this PR label Mar 10, 2023
@nima-karimi nima-karimi requested a review from a team March 10, 2023 16:17
@nima-karimi nima-karimi self-assigned this Mar 10, 2023
Copy link
Contributor

@rawdreeg rawdreeg left a comment

Choose a reason for hiding this comment

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

Thanks, @nima-karimi.

Note: I did encounter the PHP Warning: Undefined variable $cache_key .... This should be resolved by #704

LGTM! 👍🏾

@nima-karimi nima-karimi merged commit 307aa01 into feature/mcm-integration Mar 17, 2023
@nima-karimi nima-karimi deleted the feature/mcm-integration-sync-status-report branch March 17, 2023 12:18
@ecgan ecgan linked an issue May 25, 2023 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog: none Skip changelog entry for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create method(s) to report the product syncing status
2 participants