google-listings-and-ads icon indicating copy to clipboard operation
google-listings-and-ads copied to clipboard

Swap and rename dashboard performance cards

Open joemcgill opened this issue 1 year ago • 1 comments

Part of https://github.com/woocommerce/google-listings-and-ads/issues/2460

The current dashboard includes two SummaryCard components for "Performance (Free Listing)" (free listings) and "Performance (Paid Campaigns)" (paid campaigns). See current screenshot:

Image

In order to better highlight the value of setting up paid ads, we'll swap these two cards and rename them to “Google Ads” (paid campaigns) and “Free Listings (Limited Visibility)” (free-listings)

Acceptance Criteria

On the dashboard:

  • [ ] The first SummaryCard should be titled "Google Ads" and show the PaidPerformanceCard when the ads setup is complete, or PaidCampaignPromotionCard when not.
  • [ ] The second SummaryCard should be titled "Free Listings (Limited Visibility)" and show the FreePerformanceCard component.

Implementation Brief

This PR should be merged into the feature/2460-google-ads-value-prop branch.

The file where these can be swapped and renamed is found here.

Test Coverage

  • Jest tests in js/src/dashboard/summary-section/summary-section.test.js should still pass.
  • E2E tests that rely on the "Add paid campaign" button in the PaidCampaignPromotionCard component should still pass after swapping positions. If not, update the tests so they aren't dependent on the position of the SummaryCard.

Definition Questions

joemcgill avatar Aug 16 '24 21:08 joemcgill

This looks good for the Definition phase.

eason9487 avatar Aug 23 '24 10:08 eason9487

Closing this as completed since it was part of the 2.9 release.

mikkamp avatar Dec 02 '24 15:12 mikkamp