Skip to content

chore(studio): remove privacy policy update notification banner#44380

Merged
pamelachia merged 1 commit intomasterfrom
chore/remove-privacy-policy-banner
Mar 31, 2026
Merged

chore(studio): remove privacy policy update notification banner#44380
pamelachia merged 1 commit intomasterfrom
chore/remove-privacy-policy-banner

Conversation

@pamelachia
Copy link
Copy Markdown
Contributor

@pamelachia pamelachia commented Mar 31, 2026

Summary

Remove the privacy policy update notification banner that was added for the March 2026 policy update. The effective date has passed and users have had sufficient notice.

Changes

  • Remove PrivacyUpdateBanner component from AnalyticsSettings.tsx and its unused imports
  • Remove banner usage from the organizations list page and org projects page
  • Remove PRIVACY_NOTICE_ACKNOWLEDGED localStorage key

Testing

No setup needed - this is a removal of UI elements.

Tested on Vercel preview:

  • Organizations page loads without the banner
  • Org projects page loads without the banner
  • No console errors on either page
  • Analytics settings page still renders correctly

Linear

  • fixes GROWTH-692

@pamelachia pamelachia requested a review from a team as a code owner March 31, 2026 07:41
Copilot AI review requested due to automatic review settings March 31, 2026 07:41
@pamelachia pamelachia self-assigned this Mar 31, 2026
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 31, 2026

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

Project Deployment Actions Updated (UTC)
design-system Ready Ready Preview, Comment Mar 31, 2026 7:46am
docs Ready Ready Preview, Comment, Open in v0 Mar 31, 2026 7:46am
learn Ready Ready Preview, Comment Mar 31, 2026 7:46am
studio-self-hosted Ready Ready Preview, Comment Mar 31, 2026 7:46am
studio-staging Ready Ready Preview, Comment Mar 31, 2026 7:46am
ui-library Ready Ready Preview, Comment Mar 31, 2026 7:46am
zone-www-dot-com Ready Ready Preview, Comment, Open in v0 Mar 31, 2026 7:46am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
studio Ignored Ignored Mar 31, 2026 7:46am

Request Review

@supabase
Copy link
Copy Markdown

supabase bot commented Mar 31, 2026

This pull request has been ignored for the connected project xguihxuzqibwxjnimxev because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes the March 2026 privacy policy update notification banner and its persistence key now that the effective date has passed.

Changes:

  • Deletes the PrivacyUpdateBanner component from AnalyticsSettings.tsx and removes related imports.
  • Removes the banner from the organizations list page and the org projects page.
  • Removes the PRIVACY_NOTICE_ACKNOWLEDGED localStorage key constant.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
packages/common/constants/local-storage.ts Removes the PRIVACY_NOTICE_ACKNOWLEDGED key from LOCAL_STORAGE_KEYS.
apps/studio/pages/organizations.tsx Removes PrivacyUpdateBanner import and render on the organizations page.
apps/studio/pages/org/[slug]/index.tsx Removes PrivacyUpdateBanner import and render on the org projects page.
apps/studio/components/interfaces/Account/Preferences/AnalyticsSettings.tsx Deletes the PrivacyUpdateBanner component and associated imports, leaving analytics settings intact.

@github-actions
Copy link
Copy Markdown
Contributor

🎭 Playwright Test Results

passed  176 passed
skipped  4 skipped

Details

stats  180 tests across 20 suites
duration  4 minutes, 9 seconds
commit  3a80e14

Skipped tests

Features › sql-editor.spec.ts › SQL Editor › snippet favourite works as expected
Features › sql-editor.spec.ts › SQL Editor › share with team works as expected
Features › sql-editor.spec.ts › SQL Editor › folders works as expected
Features › sql-editor.spec.ts › SQL Editor › other SQL snippets actions work as expected

@pamelachia pamelachia merged commit 8364ffb into master Mar 31, 2026
36 of 38 checks passed
@pamelachia pamelachia deleted the chore/remove-privacy-policy-banner branch March 31, 2026 11:54
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 31, 2026

Braintrust eval report

Assistant (master-1774958150)

Score Average Improvements Regressions
Completeness 100% (+0pp) - -
Conciseness 28.2% (+6pp) 4 🟢 1 🔴
Docs Faithfulness 81.6% (-3pp) - 1 🔴
Goal Completion 92.3% (+0pp) - -
Tool Usage 80.6% (+0pp) - -
Correctness 100% (+0pp) - -
URL Validity 100% (+0pp) - -
SQL Identifier Quoting 100% (+0pp) - -
SQL Validity 90.9% (+18pp) 1 🟢 -
Time_to_first_token 0.69tok (+0.1tok) 5 🟢 8 🔴
Llm_calls 11.08 (-0.03) 3 🟢 6 🔴
Tool_calls 2.21 (+0.08) 4 🟢 3 🔴
Errors 0 (+0) - -
Llm_errors 0 (+0) - -
Tool_errors 0 (+0) - -
Prompt_tokens 30496.26tok (+2812.26tok) 8 🟢 4 🔴
Prompt_cached_tokens 17355.49tok (+1483.49tok) 5 🟢 4 🔴
Prompt_cache_creation_tokens 0tok (+0tok) - -
Completion_tokens 449.36tok (-25.44tok) 5 🟢 7 🔴
Completion_reasoning_tokens 57.77tok (+6.74tok) 4 🟢 8 🔴
Completion_accepted_prediction_tokens 0tok (+0tok) - -
Completion_rejected_prediction_tokens 0tok (+0tok) - -
Completion_audio_tokens 0tok (+0tok) - -
Total_tokens 30945.62tok (+2786.82tok) 7 🟢 5 🔴
Estimated_cost 0$ (+0$) 6 🟢 3 🔴
Duration 7.83s (+0.34s) 4 🟢 9 🔴
Llm_duration 13.02s (+0.39s) 4 🟢 9 🔴

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.

4 participants