Skip to content

feat: WF-2856-cookie-banner-copy integration#509

Draft
pie-design-system-app[bot] wants to merge 2 commits into
mainfrom
WF-2856-cookie-banner-copy
Draft

feat: WF-2856-cookie-banner-copy integration#509
pie-design-system-app[bot] wants to merge 2 commits into
mainfrom
WF-2856-cookie-banner-copy

Conversation

@pie-design-system-app

@pie-design-system-app pie-design-system-app Bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

This PR has been automatically generated as part of the test-aperture workflow from the PIE monorepo.

Updated the cookie banner test-bed in the v15 app to cover the new personalized copy feature from the PR.

What changed:

  • Simplified nextjs-app-v15/src/app/components/cookie-banner/cookie-banner.tsx back to the typed React wrapper instead of the imperative ref/useEffect web-component wiring.
  • Added dedicated routes for the overrides:
    • /components/cookie-banner/personalized-label
    • /components/cookie-banner/personalized-description
  • Kept the visual Percy scope narrow by removing cookie-banner-specific additions from the visual spec.

How to test locally:

  • Start or use the v15 app on http://localhost:3004
  • Open /components/cookie-banner for default copy
  • Open /components/cookie-banner/personalized-label for the label override
  • Open /components/cookie-banner/personalized-description for the description override

Verification:

  • Ran: APP_NAME=nextjs-app-v15 npx playwright test -c playwright.config.ts test/system/cookie-banner.spec.ts --project=system
  • Result: 3 passed

pie-pr-number: 2965
pie-pr-sha: ebc8cbffebe2745e30d44fe67e4402d37a663811
@pie-design-system-app
pie-design-system-app Bot force-pushed the WF-2856-cookie-banner-copy branch from de74f8f to 9f90057 Compare June 30, 2026 11:49
@github-actions
github-actions Bot temporarily deployed to nuxt-app-pr-509 June 30, 2026 11:52 Inactive
@github-actions
github-actions Bot temporarily deployed to vanilla-app-pr-509 June 30, 2026 11:52 Inactive
@github-actions
github-actions Bot temporarily deployed to nextjs-app-v14-pr-509 June 30, 2026 11:52 Inactive
@github-actions
github-actions Bot temporarily deployed to nextjs-app-v15-pr-509 June 30, 2026 11:52 Inactive
@gundabathula-jatin
gundabathula-jatin force-pushed the WF-2856-cookie-banner-copy branch from a8e29b2 to 88922dd Compare June 30, 2026 13:28
@gundabathula-jatin
gundabathula-jatin force-pushed the WF-2856-cookie-banner-copy branch from 88922dd to d53cd3b Compare June 30, 2026 13:33
@pie-design-system-app

Copy link
Copy Markdown
Contributor Author

Preview branch deleted due to inactivity (no Amplify build in the last 14 days). Push a commit or re-run CI to redeploy.

3 similar comments
@pie-design-system-app

Copy link
Copy Markdown
Contributor Author

Preview branch deleted due to inactivity (no Amplify build in the last 14 days). Push a commit or re-run CI to redeploy.

@pie-design-system-app

Copy link
Copy Markdown
Contributor Author

Preview branch deleted due to inactivity (no Amplify build in the last 14 days). Push a commit or re-run CI to redeploy.

@pie-design-system-app

Copy link
Copy Markdown
Contributor Author

Preview branch deleted due to inactivity (no Amplify build in the last 14 days). Push a commit or re-run CI to redeploy.

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.

0 participants