Merged
Conversation
* Update README.md - Added noification for ownership transfer (#2910) Summary: This is a update to notify developers that the facebook-for-woocommerce plugin is now part of Meta and development is taken care by the engineers at Meta. We also have encouraged for contributions on this repo. ### Changes proposed in this Pull Request: This PR contains mostly the changes around **README.md**. Since mostly developers of any kind prefer looking at the **README** of a GitHub repo. We have now changed the README emphasising the changes of the new **Owneship of the plugin** by Meta. And the invitation for **opensource community** for their contributions. ### Screenshots: {F1975511490} Pull Request resolved: #2910 Reviewed By: mshymon, vinkmeta Differential Revision: D70323913 Pulled By: SayanPandey fbshipit-source-id: 3fd9937d20f968132dba4fcc23ebfcabae9a3f55 * Changing APP to PLUGIN - README.MD (#2916) Summary: Changing the app word to plugin to more suit with the idea of facebook-for-woocommerce ### Changes proposed in this Pull Request: This is a simple change of APP to PLUGIN in README.md text . Pull Request resolved: #2916 Reviewed By: crisojog, vinkmeta Differential Revision: D70486710 Pulled By: SayanPandey fbshipit-source-id: 47eb9d84a32712d678e153d5b1a98fbf7074af98 * Removed the redundant entries from readme.txt and changelog.txt * Updating version to 3.4.2 --------- Co-authored-by: Sayan Pandey <sayanpandey@meta.com> Co-authored-by: Vahid Kaykhaei <vahidkay@meta.com>
Summary: ### Changes proposed in this Pull Request: This PR introduces unit tests for the Request and Response classes in the WooCommerce\Facebook\API\ProductCatalog\ItemsBatch\Create namespace. These tests ensure that the classes behave as expected when handling API requests and responses for creating items batches. - [x] Do the changed files pass `phpcs` checks? Please remove `phpcs:ignore` comments in changed files and fix any issues, or delete if not practical. ### Screenshots:  ### Detailed test instructions: 1. Run the unit tests with "./vendor/bin/phpunit tests/Unit/Api/ProductCatalog/ItemsBatch/Create" to verify that they pass Pull Request resolved: #2917 Test Plan: Imported from GitHub, without a `Test Plan:` line. **!---- (auto-generated) DO NOT EDIT OR PUT ANYTHING AFTER THIS LINE ----!** MFTRunTestsScript Run / Test Suite: sa_checkout / Test Collection: bloks / Diff Version V1 https://internalfb.com/intern/testinfra/testrun/5348024825824290 MFTRunTestsScript Run / Test Suite: sa_checkout / Test Collection: www / Diff Version V1 https://internalfb.com/intern/testinfra/testrun/1407375288546156 Reviewed By: carterbuce, ajello-meta Differential Revision: D70511474 Pulled By: nrostrow-meta fbshipit-source-id: dc57706d284bc8b0ed7a662200d39cd596c97fdb
Summary: In PRs that don't modify any php files, the "Code sniff" and "PHP Unit Tests" checks get stuck in a pending state (ex. #2918). They do not run because the PR doesn't match any of the filters that the workflows define. This causes issues because our repository is set up to require all checks to pass before merging. This PR updates the filtering on these workflows to run on every PR, regardless of what files were changed. Pull Request resolved: #2926 Reviewed By: nrostrow-meta Differential Revision: D70673188 Pulled By: carterbuce fbshipit-source-id: e2fc616d497e21b9b4f993da365afcc4a0664b4f
…ultiple colors separated by a coma (#2947) Summary: ### Changes proposed in this Pull Request: When rolling out product sync via Feeds, we noticed a fatal upload error for products that have value of a color to be multiple colors written with commas (e.g. color value: Yellow,Blue,Red). This case right leads to fatal error "feed_column_count_mismatch" on feed upload and the product is deleted from the catalog. The fix is simple what we already fo for other fields like `name` or `description`: wrap it up with quotes by calling `format_string_for_feed`. Applying this not only to color but some other fields too. Manually tested feed generation and feed upload success. ### Additional details: Screenshot shows multi color value for a single product written via comas that leads to feed sync issue before this fix. <img width="1857" alt="Screenshot 2025-03-14 at 17 10 11" src="https://github.com/user-attachments/assets/1fdffc98-b245-4c86-a479-1688a2bd2aa0" /> ### Detailed test instructions: 1. Create or update an existing product to have multi color value written via comas, as per screenshot. 2. Trigger feed generation and check for errors. ### Additional details: <!-- Optional. Enter a summary of all changes in this Pull Request, which will be added to the changelog if accepted. Each line should start with change type prefix`(Fix|Add|…) - `, for example: > Break - A change breaking previous API or functionality. > Add - A new feature, function or functionality was added. > Update - Big changes to something that wasn't broken. > Fix - Took care of something that wasn't working. > Tweak - Small change, that isn't actually very important. > Dev - Developer-facing only change. > Doc - Updated customer or developer facing documentation If you remove the "Changelog entry" header, the Pull Request title will be used as the changelog entry. Add the `changelog: none` label if no changelog entry is needed. --> ### Changelog entry > Fix - feed upload error "feed_column_count_mismatch" when product has multiple colors separated by a coma. Pull Request resolved: #2947 Reviewed By: vinkmeta Differential Revision: D71206033 Pulled By: mshymon fbshipit-source-id: da898f1bf0d3cb701098bb1bc72f85942d97675d
Summary: ### Changes proposed in this Pull Request: Updated Pull Request Template ### Changelog entry > Updated Pull Request Template Pull Request resolved: #2948 Reviewed By: sol-loup, ajello-meta, jczhuoMeta, gurtejrehal Differential Revision: D71211859 Pulled By: vinkmeta fbshipit-source-id: 45992254125bfc3ce3c267d7cb3afd1a73d0bb01
…2889) Summary: This code change should have no functional effect, just updating code of the function prepare_product() in fbproduct.php to improve readability of seeing difference of data population depending on `$type_to_prepare_for` function param value. 1. Unit tests: `npm run test:php ` 2. Manual smoke testing. <!-- Optional. Enter a summary of all changes in this Pull Request, which will be added to the changelog if accepted. Each line should start with change type prefix`(Fix|Add|…) - `, for example: > Break - A change breaking previous API or functionality. > Add - A new feature, function or functionality was added. > Update - Big changes to something that wasn't broken. > Fix - Took care of something that wasn't working. > Tweak - Small change, that isn't actually very important. > Dev - Developer-facing only change. > Doc - Updated customer or developer facing documentation If you remove the "Changelog entry" header, the Pull Request title will be used as the changelog entry. Add the `changelog: none` label if no changelog entry is needed. --> > Dev - Improved readability of function prepare_product() in fbproduct.php Pull Request resolved: #2889 Reviewed By: devbodaghe Differential Revision: D69792569 Pulled By: mshymon fbshipit-source-id: 2dc58c69baadb4ee684cf4047e666c29dd191a7f
Summary: The main change of this PR is adding a <coverage> section in phpunit.xml.dist. This change, along with a local installation of a code coverage driver (xdebug), are required to get the coverage report working. - [x] Do the changed files pass `phpcs` checks? Please remove `phpcs:ignore` comments in changed files and fix any issues, or delete if not practical. <img width="1524" alt="image" src="https://github.com/user-attachments/assets/870b0fc3-bf0d-4eeb-b956-e0f8e1e0b870" /> 1. pecl install xdebug-3.1.6 2. Add xdebug.mode = coverage to php.ini or set environment variable XDEBUG_MODE=coverage 3. `./vendor/bin/phpunit --coverage-html=reports/coverage` 4. open reports/coverage/index.html in web browser > Enabled PHPUnit code coverage report generation Pull Request resolved: #2893 Reviewed By: Jmencab Differential Revision: D70021799 Pulled By: carterbuce fbshipit-source-id: 6a4c5204bbbf512b38c75f6aa1a820b3514a58d8
) Summary: Bumps [tj-actions/changed-files](https://github.com/tj-actions/changed-files) from 41 to 46. <details> <summary>Release notes</summary> <p><em>Sourced from <a href="https://github.com/tj-actions/changed-files/releases">tj-actions/changed-files's releases</a>.</em></p> <blockquote> <h2>v46</h2> <blockquote> <p>[!WARNING]<br /> <strong>Security Alert:</strong> A critical security issue was identified in this action due to a compromised commit.</p> <p>This commit has been <strong>removed</strong> from all tags and branches, and necessary measures have been implemented to prevent similar issues in the future.</p> <h4><strong>Action Required:</strong></h4> <ul> <li><strong>Review your workflows executed between March 14 and March 15.</strong> If you notice unexpected output under the <code>changed-files</code> section, decode it using the following command: <code>echo 'xxx' | base64 -d | base64 -d</code><br /> If the output contains sensitive information (e.g., tokens or secrets), <strong>revoke and rotate those secrets immediately</strong>.</li> <li><strong>If your workflows reference this commit directly by its SHA</strong>, you must update them immediately to avoid using the compromised version.</li> <li><strong>If you are using tagged versions</strong> (e.g., <code>v35</code>, <code>v44.5.1</code>), no action is required as these tags have been updated and are now safe to use.</li> </ul> <p>Additionally, as a precaution, we recommend rotating any secrets that may have been exposed during this timeframe to ensure the continued security of your workflows.</p> </blockquote> <h1>Changes in v46.0.1</h1> <h2>What's Changed</h2> <ul> <li>update: sync-release-version.yml to use signed commits by <a href="https://github.com/jackton1"><code>@jackton1</code></a> in <a href="https://github.com/tj-actions/changed-files/pull/2472">tj-actions/changed-files#2472</a></li> <li>Updated README.md by <a href="https://github.com/github-actions"><code>@github-actions</code></a> in <a href="https://github.com/tj-actions/changed-files/pull/2473">tj-actions/changed-files#2473</a></li> </ul> <p><strong>Full Changelog</strong>: <a href="https://github.com/tj-actions/changed-files/compare/v46...v46.0.1">https://github.com/tj-actions/changed-files/compare/v46...v46.0.1</a></p> <hr /> <h1>Changes in v46.0.0</h1> <h2>What's Changed</h2> <ul> <li>docs: update docs to highlight security issues by <a href="https://github.com/jackton1"><code>@jackton1</code></a> in <a href="https://github.com/tj-actions/changed-files/pull/2465">tj-actions/changed-files#2465</a></li> <li>fix: update github workflow update-readme.yml by <a href="https://github.com/jackton1"><code>@jackton1</code></a> in <a href="https://github.com/tj-actions/changed-files/pull/2466">tj-actions/changed-files#2466</a></li> <li>fix: update permission in update-readme.yml workflow by <a href="https://github.com/jackton1"><code>@jackton1</code></a> in <a href="https://github.com/tj-actions/changed-files/pull/2467">tj-actions/changed-files#2467</a></li> <li>fix: update update-readme.yml to sign-commits by <a href="https://github.com/jackton1"><code>@jackton1</code></a> in <a href="https://github.com/tj-actions/changed-files/pull/2468">tj-actions/changed-files#2468</a></li> <li>Updated README.md by <a href="https://github.com/github-actions"><code>@github-actions</code></a> in <a href="https://github.com/tj-actions/changed-files/pull/2469">tj-actions/changed-files#2469</a></li> <li>update: sync-release-version.yml by <a href="https://github.com/jackton1"><code>@jackton1</code></a> in <a href="https://github.com/tj-actions/changed-files/pull/2471">tj-actions/changed-files#2471</a></li> </ul> <h2>New Contributors</h2> <ul> <li><a href="https://github.com/github-actions"><code>@github-actions</code></a> made their first contribution in <a href="https://github.com/tj-actions/changed-files/pull/2469">tj-actions/changed-files#2469</a></li> </ul> <p><strong>Full Changelog</strong>: <a href="https://github.com/tj-actions/changed-files/compare/v45.0.5...v46.0.0">https://github.com/tj-actions/changed-files/compare/v45.0.5...v46.0.0</a></p> <h2>What's Changed</h2> <ul> <li>docs: update docs to highlight security issues by <a href="https://github.com/jackton1"><code>@jackton1</code></a> in <a href="https://github.com/tj-actions/changed-files/pull/2465">tj-actions/changed-files#2465</a></li> <li>fix: update github workflow update-readme.yml by <a href="https://github.com/jackton1"><code>@jackton1</code></a> in <a href="https://github.com/tj-actions/changed-files/pull/2466">tj-actions/changed-files#2466</a></li> <li>fix: update permission in update-readme.yml workflow by <a href="https://github.com/jackton1"><code>@jackton1</code></a> in <a href="https://github.com/tj-actions/changed-files/pull/2467">tj-actions/changed-files#2467</a></li> <li>fix: update update-readme.yml to sign-commits by <a href="https://github.com/jackton1"><code>@jackton1</code></a> in <a href="https://github.com/tj-actions/changed-files/pull/2468">tj-actions/changed-files#2468</a></li> <li>Updated README.md by <a href="https://github.com/github-actions"><code>@github-actions</code></a> in <a href="https://github.com/tj-actions/changed-files/pull/2469">tj-actions/changed-files#2469</a></li> <li>update: sync-release-version.yml by <a href="https://github.com/jackton1"><code>@jackton1</code></a> in <a href="https://github.com/tj-actions/changed-files/pull/2471">tj-actions/changed-files#2471</a></li> </ul> <p><strong>Full Changelog</strong>: <a href="https://github.com/tj-actions/changed-files/compare/v45.0.5...v46.0.0">https://github.com/tj-actions/changed-files/compare/v45.0.5...v46.0.0</a></p> </blockquote> <p>... (truncated)</p> </details> <details> <summary>Changelog</summary> <p><em>Sourced from <a href="https://github.com/tj-actions/changed-files/blob/main/HISTORY.md">tj-actions/changed-files's changelog</a>.</em></p> <blockquote> <h1>Changelog</h1> <h1><a href="https://github.com/tj-actions/changed-files/compare/v46.0.0...v46.0.1">46.0.1</a> - (2025-03-16)</h1> <h2><!-- raw HTML omitted -->🔄 Update</h2> <ul> <li>Updated README.md (<a href="https://github.com/tj-actions/changed-files/issues/2473">https://github.com/facebook/facebook-for-woocommerce/issues/2473</a>)</li> </ul> <p>Co-authored-by: github-actions[bot] <!-- raw HTML omitted --> (<a href="https://github.com/tj-actions/changed-files/commit/2f7c5bfce28377bc069a65ba478de0a74aa0ca32">2f7c5bf</a>) - (github-actions[bot])</p> <ul> <li>Sync-release-version.yml to use signed commits (<a href="https://github.com/tj-actions/changed-files/issues/2472">https://github.com/facebook/facebook-for-woocommerce/issues/2472</a>) (<a href="https://github.com/tj-actions/changed-files/commit/4189ec62c445484531e9ad97157d990be96e88ee">4189ec6</a>) - (Tonye Jack)</li> </ul> <h1><a href="https://github.com/tj-actions/changed-files/compare/v45.0.9...v46.0.0">46.0.0</a> - (2025-03-16)</h1> <h2><!-- raw HTML omitted -->🐛 Bug Fixes</h2> <ul> <li>Update update-readme.yml to sign-commits (<a href="https://github.com/tj-actions/changed-files/issues/2468">https://github.com/facebook/facebook-for-woocommerce/issues/2468</a>) (<a href="https://github.com/tj-actions/changed-files/commit/0f1ffe61855cb317d5fd66122c14dc0627eab141">0f1ffe6</a>) - (Tonye Jack)</li> <li>Update permission in update-readme.yml workflow (<a href="https://github.com/tj-actions/changed-files/issues/2467">https://github.com/facebook/facebook-for-woocommerce/issues/2467</a>) (<a href="https://github.com/tj-actions/changed-files/commit/ddef03e37c84cfb9ee89fa055b86359aaf949c86">ddef03e</a>) - (Tonye Jack)</li> <li>Update github workflow update-readme.yml (<a href="https://github.com/tj-actions/changed-files/issues/2466">#2466</a>) (<a href="https://github.com/tj-actions/changed-files/commit/9c2df0d54a911c819d7368d7e5ed7c01c0796e0a">9c2df0d</a>) - (Tonye Jack)</li> </ul> <h2><!-- raw HTML omitted -->➖ Remove</h2> <ul> <li>Deleted renovate.json (<a href="https://github.com/tj-actions/changed-files/commit/e37e952786556966c1fb6183c5937b3966bab099">e37e952</a>) - (Tonye Jack)</li> </ul> <h2><!-- raw HTML omitted -->🔄 Update</h2> <ul> <li>Sync-release-version.yml (<a href="https://github.com/tj-actions/changed-files/issues/2471">https://github.com/facebook/facebook-for-woocommerce/issues/2471</a>) (<a href="https://github.com/tj-actions/changed-files/commit/4cd184a1dd542b79cca1d4d7938e4154a6520ca7">4cd184a</a>) - (Tonye Jack)</li> <li>Updated README.md (<a href="https://github.com/tj-actions/changed-files/issues/2469">https://github.com/facebook/facebook-for-woocommerce/issues/2469</a>)</li> </ul> <p>Co-authored-by: github-actions[bot] <!-- raw HTML omitted --> (<a href="https://github.com/tj-actions/changed-files/commit/5cbf22026d05fbef0c027d1b1f118fe3a1b6e435">5cbf220</a>) - (github-actions[bot])</p> <h2><!-- raw HTML omitted -->📚 Documentation</h2> <ul> <li>Update docs to highlight security issues (<a href="https://github.com/tj-actions/changed-files/issues/2465">https://github.com/facebook/facebook-for-woocommerce/issues/2465</a>) (<a href="https://github.com/tj-actions/changed-files/commit/65253327cf47481b4b1b4b9fea78e143a1353147">6525332</a>) - (Tonye Jack)</li> </ul> <h1><a href="https://github.com/tj-actions/changed-files/compare/v45.0.4...v45.0.9">45.0.9</a> - (2025-03-15)</h1> <h2><!-- raw HTML omitted -->🐛 Bug Fixes</h2> <ul> <li><strong>deps:</strong> Update dependency <code>@octokit/rest</code> to v21.1.1 (<a href="https://github.com/tj-actions/changed-files/issues/2435">https://github.com/facebook/facebook-for-woocommerce/issues/2435</a>) (<a href="https://github.com/tj-actions/changed-files/commit/fb8dcda5fb8954cec37773d2b275a8579c86c781">fb8dcda</a>) - (renovate[bot])</li> <li><strong>deps:</strong> Update dependency <code>@octokit/rest</code> to v21.1.0 (<a href="https://github.com/tj-actions/changed-files/issues/2394">#2394</a>) (<a href="https://github.com/tj-actions/changed-files/commit/7b72c97d739f955f5cadca0d59799d826ae9f6c9">7b72c97</a>) - (renovate[bot])</li> <li><strong>deps:</strong> Update dependency yaml to v2.7.0 (<a href="https://github.com/tj-actions/changed-files/issues/2383">#2383</a>) (<a href="https://github.com/tj-actions/changed-files/commit/5f974c28f5044c411f0c9e7becf3f172029cf9cf">5f974c2</a>) - (renovate[bot])</li> </ul> <h2><!-- raw HTML omitted -->⚙️ Miscellaneous Tasks</h2> <ul> <li><strong>deps:</strong> Lock file maintenance (<a href="https://github.com/tj-actions/changed-files/issues/2460">https://github.com/facebook/facebook-for-woocommerce/issues/2460</a>) (<a href="https://github.com/tj-actions/changed-files/commit/9200e69727eb73eb060652b19946b8a2fdfb654b">9200e69</a>) - (renovate[bot])</li> <li><strong>deps:</strong> Update dependency <code>@types/node</code> to v22.13.10 (<a href="https://github.com/tj-actions/changed-files/issues/2459">https://github.com/facebook/facebook-for-woocommerce/issues/2459</a>) (<a href="https://github.com/tj-actions/changed-files/commit/e650cfdae513481a20f538e88d98b39106523006">e650cfd</a>) - (renovate[bot])</li> <li><strong>deps:</strong> Update dependency eslint-config-prettier to v10.1.1 (<a href="https://github.com/tj-actions/changed-files/issues/2458">https://github.com/facebook/facebook-for-woocommerce/issues/2458</a>) (<a href="https://github.com/tj-actions/changed-files/commit/82af21f4a05896ca18c950539469bee225c45a89">82af21f</a>) - (renovate[bot])</li> <li><strong>deps:</strong> Update dependency eslint-config-prettier to v10.1.0 (<a href="https://github.com/tj-actions/changed-files/issues/2457">https://github.com/facebook/facebook-for-woocommerce/issues/2457</a>) (<a href="https://github.com/tj-actions/changed-files/commit/82fa4a6402582d5c8c9c0e95b7ff7cc88992bbb4">82fa4a6</a>) - (renovate[bot])</li> <li><strong>deps:</strong> Update peter-evans/create-pull-request action to v7.0.8 (<a href="https://github.com/tj-actions/changed-files/issues/2455">https://github.com/facebook/facebook-for-woocommerce/issues/2455</a>) (<a href="https://github.com/tj-actions/changed-files/commit/315505acf41d2913b71af48080fb158cd01f79e7">315505a</a>) - (renovate[bot])</li> <li><strong>deps:</strong> Update dependency <code>@types/node</code> to v22.13.9 (<a href="https://github.com/tj-actions/changed-files/issues/2454">https://github.com/facebook/facebook-for-woocommerce/issues/2454</a>) (<a href="https://github.com/tj-actions/changed-files/commit/c8e1cdb9ea135ee549963c167ffaec5e7d4a71cd">c8e1cdb</a>) - (renovate[bot])</li> </ul> </blockquote> <p>... (truncated)</p> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/tj-actions/changed-files/commit/2f7c5bfce28377bc069a65ba478de0a74aa0ca32"><code>2f7c5bf</code></a> Updated README.md (<a href="https://github.com/tj-actions/changed-files/issues/2473">https://github.com/facebook/facebook-for-woocommerce/issues/2473</a>)</li> <li><a href="https://github.com/tj-actions/changed-files/commit/4189ec62c445484531e9ad97157d990be96e88ee"><code>4189ec6</code></a> update: sync-release-version.yml to use signed commits (<a href="https://github.com/tj-actions/changed-files/issues/2472">https://github.com/facebook/facebook-for-woocommerce/issues/2472</a>)</li> <li><a href="https://github.com/tj-actions/changed-files/commit/4cd184a1dd542b79cca1d4d7938e4154a6520ca7"><code>4cd184a</code></a> update: sync-release-version.yml (<a href="https://github.com/tj-actions/changed-files/issues/2471">https://github.com/facebook/facebook-for-woocommerce/issues/2471</a>)</li> <li><a href="https://github.com/tj-actions/changed-files/commit/5cbf22026d05fbef0c027d1b1f118fe3a1b6e435"><code>5cbf220</code></a> Updated README.md (<a href="https://github.com/tj-actions/changed-files/issues/2469">https://github.com/facebook/facebook-for-woocommerce/issues/2469</a>)</li> <li><a href="https://github.com/tj-actions/changed-files/commit/0f1ffe61855cb317d5fd66122c14dc0627eab141"><code>0f1ffe6</code></a> fix: update update-readme.yml to sign-commits (<a href="https://github.com/tj-actions/changed-files/issues/2468">https://github.com/facebook/facebook-for-woocommerce/issues/2468</a>)</li> <li><a href="https://github.com/tj-actions/changed-files/commit/ddef03e37c84cfb9ee89fa055b86359aaf949c86"><code>ddef03e</code></a> fix: update permission in update-readme.yml workflow (<a href="https://github.com/tj-actions/changed-files/issues/2467">https://github.com/facebook/facebook-for-woocommerce/issues/2467</a>)</li> <li><a href="https://github.com/tj-actions/changed-files/commit/9c2df0d54a911c819d7368d7e5ed7c01c0796e0a"><code>9c2df0d</code></a> fix: update github workflow update-readme.yml (<a href="https://github.com/tj-actions/changed-files/issues/2466">#2466</a>)</li> <li><a href="https://github.com/tj-actions/changed-files/commit/65253327cf47481b4b1b4b9fea78e143a1353147"><code>6525332</code></a> docs: update docs to highlight security issues (<a href="https://github.com/tj-actions/changed-files/issues/2465">https://github.com/facebook/facebook-for-woocommerce/issues/2465</a>)</li> <li><a href="https://github.com/tj-actions/changed-files/commit/e37e952786556966c1fb6183c5937b3966bab099"><code>e37e952</code></a> Deleted renovate.json</li> <li><a href="https://github.com/tj-actions/changed-files/commit/a284dc1814e3fd07f2e34267fc8f81227ed29fb8"><code>a284dc1</code></a> Upgraded to v45.0.8 (<a href="https://github.com/tj-actions/changed-files/issues/2462">https://github.com/facebook/facebook-for-woocommerce/issues/2462</a>)</li> <li>Additional commits viewable in <a href="https://github.com/tj-actions/changed-files/compare/v41...v46">compare view</a></li> </ul> </details> <br /> [](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- <details> <summary>Dependabot commands and options</summary> <br /> You can trigger Dependabot actions by commenting on this PR: - `dependabot rebase` will rebase this PR - `dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `dependabot merge` will merge this PR after your CI passes on it - `dependabot squash and merge` will squash and merge this PR after your CI passes on it - `dependabot cancel merge` will cancel a previously requested merge and block automerging - `dependabot reopen` will reopen this PR if it is closed - `dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency - `dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) You can disable automated security fix PRs for this repo from the [Security Alerts page](https://github.com/facebook/facebook-for-woocommerce/network/alerts). </details> Pull Request resolved: #2951 Reviewed By: sdwilsh Differential Revision: D71329113 Pulled By: bigfootjon fbshipit-source-id: ff3ddd7fe208ad90c68967169f9443a066231b4e
vahidkay-meta
force-pushed
the
release/3.4.3
branch
from
36584ff to
bcf0f60
Compare
tzahgr
approved these changes
Mar 24, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add - Items batch request and response tests by @nrostrow-meta in #2917
Tweak - Always run PHP-based github workflows by @carterbuce in #2926
Fix - feed upload error "feed_column_count_mismatch" when product has multiple colors separated by a comma by @mshymon in #2947
Tweak - Updated Pull Request Template by @vinkmeta in #2948
Tweak - Enabled PHPUnit code coverage report generation by @carterbuce in #2893
Dev - Improved readability of function prepare_product() in fbproduct.php by @mshymon in #2889