Skip to content

GRWMK-494: Add Consent Banner - #493

Merged
angelazhou32 merged 3 commits into
mainfrom
grwmk-494/add-consent-banner
Sep 16, 2026
Merged

GRWMK-494: Add Consent Banner#493
angelazhou32 merged 3 commits into
mainfrom
grwmk-494/add-consent-banner

Conversation

@davidcarlsonberg

@davidcarlsonberg davidcarlsonberg commented Sep 15, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

What was changed

  • Pull in consent-banner package at the Root level.
  • Integrate consent choice options, regime, and geo functionality into Learn.
  • Refresh the dependency lockfile after a stale Yarn integrity mismatch blocked install/build validation in CI and Vercel.
  • Some unrelated formatting occurred in touched files.

Why?

  • Learn does not have proper consent gating.
  • The dependency lock refresh was necessary because the existing lockfile had stale/inconsistent registry integrity metadata, which caused yarn install to fail before the app could build.

Checklist

  1. Closes GRWMK-494.

  2. How was this tested:

  • Verified the project still builds locally via yarn build.
  • Re-ran dependency installation after refreshing the lockfile to confirm the install resolves cleanly and the lockfile is consistent again.
  1. Any docs updates needed?
  • No docs updates are required for this change.

@davidcarlsonberg davidcarlsonberg self-assigned this Sep 15, 2026
@vercel

vercel Bot commented Sep 15, 2026 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
temporal-learning Ready Ready Preview Sep 15, 2026 4:05pm UTC

Request Review

@davidcarlsonberg
davidcarlsonberg marked this pull request as ready for review September 15, 2026 16:03
@davidcarlsonberg
davidcarlsonberg requested a review from a team as a code owner September 15, 2026 16:03
angelazhou32
angelazhou32 approved these changes Sep 16, 2026
View reviewed changes

@angelazhou32 angelazhou32 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you for this!

@angelazhou32
angelazhou32 merged commit 03a18b6 into main Sep 16, 2026
6 checks passed

This branch was successfully deployed

1 active deployment
Preview — b6634be9 Deployed Sep 15, 2026 by vercel[bot]
View deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@angelazhou32 angelazhou32 angelazhou32 approved these changes

Assignees

@davidcarlsonberg davidcarlsonberg

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@davidcarlsonberg @angelazhou32