Skip to content

Feature: In-Product Placements#3307

Open
asvinb wants to merge 115 commits intodevelopfrom
feature/in-product-placements
Open

Feature: In-Product Placements#3307
asvinb wants to merge 115 commits intodevelopfrom
feature/in-product-placements

Conversation

@asvinb
Copy link
Collaborator

@asvinb asvinb commented Mar 18, 2026

Changes proposed in this Pull Request:

Closes https://linear.app/a8c/project/in-product-placements-f8f73d74de8e/overview

Replace this with a good description of your changes & reasoning.

Screenshots:

Detailed test instructions:

Additional details:

Changelog entry

Add -

jamesmorrison and others added 30 commits February 9, 2026 13:07
Co-authored-by: Joe Grainger <904708+jjgrainger@users.noreply.github.com>
…ate-if-source-is-google' of github.com:woocommerce/google-listings-and-ads into feature/GOOWOO-454-add-new-property-in-glaData-to-indicate-if-source-is-google
Co-authored-by: asvinb <asvin.balloo@gmail.com> based on #3245 PoC
…der-page-only' of github.com:woocommerce/google-listings-and-ads into feature/GOOWOO-457-attribution-meta-mountpoint
…a-boxes-js-on-the-wc-edit-order-page-only

[GOOWOO-453] Load meta-boxes JS only on WC Edit Order screen
…property-in-glaData-to-indicate-if-source-is-google

[GOOWOO-454] Add new property within glaData to indicate whether source is Google.
…ion-meta-mountpoint

Feature/GOOWOO-457 Attribution order meta box mount-point
AlejandroPerezMartin and others added 22 commits February 27, 2026 12:40
…e/google-listings-and-ads into feature/in-product-placements
Update Channel Visibility meta box UI to match the latest designs
…xes-js-on-the-wc-edit-order-page-only

[GOOWOO-453] Fix meta-boxes JS only on WC Edit Order screen
…load-meta-boxes-js-on-the-wc-edit-order-page-only

Revert "[GOOWOO-453] Fix meta-boxes JS only on WC Edit Order screen"
…e/google-listings-and-ads into feature/in-product-placements
@codecov
Copy link

codecov bot commented Mar 18, 2026

Codecov Report

❌ Patch coverage is 23.07692% with 140 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.2%. Comparing base (54523c8) to head (14c2338).

Files with missing lines Patch % Lines
src/Admin/Admin.php 0.0% 91 Missing ⚠️
src/Admin/MetaBox/ChannelVisibilityMetaBox.php 0.0% 48 Missing ⚠️
views/meta-box/channel_visibility.php 0.0% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             develop   #3307     +/-   ##
===========================================
- Coverage       66.3%   66.2%   -0.1%     
- Complexity      5389    5429     +40     
===========================================
  Files            907     909      +2     
  Lines          27939   28060    +121     
  Branches        1649    1660     +11     
===========================================
+ Hits           18534   18573     +39     
- Misses          9164    9246     +82     
  Partials         241     241             
Flag Coverage Δ
js-unit-tests 64.2% <100.0%> (+0.2%) ⬆️
php-unit-tests 66.8% <7.3%> (-0.2%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...c/meta-boxes/order-attribution/google-ads-promo.js 100.0% <100.0%> (ø)
js/src/utils/urls.js 55.3% <100.0%> (ø)
src/API/Google/AdsCampaign.php 64.7% <100.0%> (+0.1%) ⬆️
src/API/Google/Query/AdsCampaignQuery.php 100.0% <100.0%> (ø)
...rc/API/Site/Controllers/Ads/CampaignController.php 100.0% <100.0%> (ø)
src/Product/ChannelVisibilityMetaBox.php 50.0% <100.0%> (ø)
views/meta-box/channel_visibility.php 0.0% <0.0%> (ø)
src/Admin/MetaBox/ChannelVisibilityMetaBox.php 0.0% <0.0%> (ø)
src/Admin/Admin.php 0.0% <0.0%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

6 participants