Skip to content

Draft: [GOOWOO-473] Ensure GenAI works together with SBM #3267

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.

Sign up for GitHub

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

Closed
jamesmorrison wants to merge 160 commits into from

Resolved PHP / composer errors.

639001e
Select commit
Loading
Failed to load commit list.
Closed

Draft: [GOOWOO-473] Ensure GenAI works together with SBM #3267

Resolved PHP / composer errors.
639001e
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Feb 24, 2026 in 0s

66.8% (+0.6%) compared to ec15530

View this Pull Request on Codecov

66.8% (+0.6%) compared to ec15530

Details

Codecov Report

❌ Patch coverage is 52.02703% with 284 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.8%. Comparing base (ec15530) to head (639001e).
⚠️ Report is 26 commits behind head on feature/GOOWOO-337-serviced-based-merchants.

Files with missing lines Patch % Lines
js/src/hooks/useCreateGenAIAssets.js 13.7% 37 Missing and 7 partials ⚠️
js/src/data/utils.js 0.0% 30 Missing and 7 partials ⚠️
js/src/components/paid-ads/campaign-assets-form.js 18.6% 31 Missing and 4 partials ⚠️
js/src/data/reducer.js 0.0% 14 Missing and 4 partials ⚠️
...s/asset-group/asset-group-header/final-url-card.js 0.0% 17 Missing ⚠️
...Site/Controllers/Ads/AssetGenerationController.php 85.2% 17 Missing ⚠️
...src/components/paid-ads/asset-group/asset-group.js 0.0% 15 Missing ⚠️
...ads/asset-group/asset-group-editor/texts-editor.js 39.1% 12 Missing and 2 partials ⚠️
...e/Controllers/MerchantCenter/AccountController.php 0.0% 14 Missing ⚠️
...up/asset-group-editor/gen-ai-image-picker/index.js 43.5% 11 Missing and 2 partials ⚠️
... and 10 more
Additional details and impacted files

Impacted file tree graph

@@                               Coverage Diff                                @@
##             feature/GOOWOO-337-serviced-based-merchants   #3267      +/-   ##
================================================================================
+ Coverage                                           66.2%   66.8%    +0.6%     
- Complexity                                             0    5172    +5172     
================================================================================
  Files                                                374     892     +518     
  Lines                                               6047   27143   +21096     
  Branches                                            1519    1630     +111     
================================================================================
+ Hits                                                4002   18124   +14122     
- Misses                                              1848    8788    +6940     
- Partials                                             197     231      +34
Files with missing lines Coverage Δ
...oup/asset-group-editor/asset-group-text-section.js 7.4% <ø> (ø)
...oup/asset-group-editor/asset-item-action-button.js 100.0% <100.0%> (ø)
...asset-group-editor/utils/fill-empty-asset-slots.js 100.0% <100.0%> (ø)
...asset-group-editor/youtube-video-selector/index.js 90.0% <100.0%> (ø)
js/src/components/paid-ads/assetSpecs.js 100.0% <ø> (ø)
js/src/components/paid-ads/gen-ai-card.js 100.0% <ø> (ø)
js/src/constants.js 100.0% <100.0%> (ø)
js/src/data/action-types.js 100.0% <ø> (ø)
js/src/hooks/useGenAIMediaAssets.js 100.0% <100.0%> (ø)
js/src/hooks/useGenAITextAssets.js 100.0% <100.0%> (ø)
... and 45 more

... and 485 files with indirect coverage changes

🚀 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.
View more details on Codecov