Skip to content

TRAC-1395: fix - treat disabled cookie consent as implicit consent server-side - #3182

Merged
parthshahp merged 1 commit into
canaryfrom
trac-1395-server-consent
Aug 25, 2026
Merged

TRAC-1395: fix - treat disabled cookie consent as implicit consent server-side#3182
parthshahp merged 1 commit into
canaryfrom
trac-1395-server-consent

Conversation

@parthshahp

Copy link
Copy Markdown
Contributor

What/Why?

Implicitly grant consent on server side requests when cookieConsent setting is disabled in the control panel.

Testing

Revert this PR.

Migration

This PR is just for sharing right now.

@changeset-bot

changeset-bot Bot commented Aug 19, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: c03ffd4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@bigcommerce/catalyst-core Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel

vercel Bot commented Aug 19, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
catalyst Ready Ready Preview Aug 25, 2026 3:18pm

Request Review

@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Bundle Size Report

Comparing against baseline from ce7d1b2 (2026-08-25).

No bundle size changes detected.

@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Unlighthouse Performance Comparison — Vercel

Comparing PR preview deployment Unlighthouse scores vs production Unlighthouse scores.

Summary Score

Aggregate score across all categories as reported by Unlighthouse.

Prod Desktop Prod Mobile Preview Desktop Preview Mobile
Score 89 92 92 95

Category Scores

Category Prod Desktop Prod Mobile Preview Desktop Preview Mobile
Performance 74 79 75 97
Accessibility 95 98 95 98
Best Practices 100 100 100 100
SEO 100 88 100 100

Core Web Vitals

Metric Prod Desktop Prod Mobile Preview Desktop Preview Mobile
LCP 3.7 s 5.6 s 3.7 s 2.6 s
CLS 0 0 0.001 0
FCP 1.2 s 1.2 s 1.2 s 1.2 s
TBT 20 ms 0 ms 20 ms 0 ms
Max Potential FID 70 ms 40 ms 70 ms 40 ms
Time to Interactive 3.7 s 5.6 s 6.5 s 2.6 s

Full Unlighthouse report →

@chanceaclark chanceaclark 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.

I'm good with this @parthshahp

@parthshahp
parthshahp marked this pull request as ready for review August 25, 2026 15:16
@parthshahp
parthshahp requested a review from a team as a code owner August 25, 2026 15:16
@parthshahp
parthshahp force-pushed the trac-1395-server-consent branch from 8002b23 to c03ffd4 Compare August 25, 2026 15:18
@parthshahp
parthshahp added this pull request to the merge queue Aug 25, 2026
@parthshahp
parthshahp removed this pull request from the merge queue due to a manual request Aug 25, 2026
@parthshahp
parthshahp added this pull request to the merge queue Aug 25, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 25, 2026
@parthshahp
parthshahp added this pull request to the merge queue Aug 25, 2026
Merged via the queue into canary with commit eb38614 Aug 25, 2026
18 of 19 checks passed
@parthshahp
parthshahp deleted the trac-1395-server-consent branch August 25, 2026 18:50
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.

2 participants