[GOOWOO-370] Product Feed showing now synced products#3303
Open
jamesmorrison wants to merge 1 commit intodevelopfrom
Open
[GOOWOO-370] Product Feed showing now synced products#3303jamesmorrison wants to merge 1 commit intodevelopfrom
jamesmorrison wants to merge 1 commit intodevelopfrom
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #3303 +/- ##
===========================================
+ Coverage 66.3% 67.1% +0.7%
+ Complexity 5389 5388 -1
===========================================
Files 907 526 -381
Lines 27939 21483 -6456
Branches 1649 0 -1649
===========================================
- Hits 18534 14409 -4125
+ Misses 9164 7074 -2090
+ Partials 241 0 -241
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
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
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.
Changes proposed in this Pull Request:
Closes https://linear.app/a8c/issue/GOOWOO-370/product-feed-showing-now-synced-products
Screenshots:
Before:

After:

Detailed test instructions:
developbranchwp post list --post_type=product --post_status=publish --fields=ID,post_title --format=table){ID}with the product ID):Note: a delete error may show for the last 2 items, it's not a fail, they just didn't exist.
wp post meta list {ID} --keys=_wc_gla_visibility,_wc_gla_sync_status,_wc_gla_synced_at,_wc_gla_google_ids,_wc_gla_mc_status --format=tableExpected: only visibility = sync-and-show, sync_status = synced, synced_at = a number. google_ids and mc_status must be absent.
Wait up to 10 seconds for the status column to populate.
Expected: products show "Not synced"
Check out this feature branch:
feature/GOOWOO-370-product-feed-showing-now-synced-productsReload the product feed page in WP Admin
Wait up to 10 seconds for the status column to populate.
Expected: products show "Pending"
"Pending" is the correct result — the product has been submitted via pull sync but Google has not yet written back a confirmed MC status. In a live connected environment this would progress to "Approved" once the MC status refresh job runs.
Additional details: