Skip to content

Only insert product summary placeholder on web#15271

Merged
emma-imber merged 2 commits intomainfrom
ei/only-render-at-a-glance-test-web
Feb 2, 2026
Merged

Only insert product summary placeholder on web#15271
emma-imber merged 2 commits intomainfrom
ei/only-render-at-a-glance-test-web

Conversation

@emma-imber
Copy link
Contributor

What does this change?

Checks that the renderingTarget is Web before adding a product summary placeholder in the at a glance redesign test.

Why?

We're still working to understand how the testing framework works with App and whether we can even collect data from it, so for now we just want to limit the test to Web.

Screenshots

Tested by adding ?dcr=apps to a test page url

Before After
Screenshot 2026-02-02 at 12 51 32 Screenshot 2026-02-02 at 12 51 47

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Hello 👋! When you're ready to run Chromatic, please apply the run_chromatic label to this PR.

You will need to reapply the label each time you want to run Chromatic.

Click here to see the Chromatic project.

@emma-imber emma-imber added run_chromatic Runs chromatic when label is applied feature Departmental tracking: work on a new feature labels Feb 2, 2026
Copy link
Contributor

@oliverabrahams oliverabrahams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Feb 2, 2026
@emma-imber emma-imber merged commit 374931a into main Feb 2, 2026
28 of 30 checks passed
@emma-imber emma-imber deleted the ei/only-render-at-a-glance-test-web branch February 2, 2026 14:25
@gu-prout
Copy link

gu-prout bot commented Feb 2, 2026

Seen on PROD (merged by @emma-imber 7 minutes and 57 seconds ago) Please check your changes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Departmental tracking: work on a new feature Seen-on-PROD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants