Skip to content

Conversation

@pamelachia
Copy link
Contributor

@pamelachia pamelachia commented Jul 31, 2025

Eliminate the Terms of Service update banner and related local storage handling from the application.

  • fixes GROWTH-447

@pamelachia pamelachia requested a review from a team as a code owner July 31, 2025 07:21
@pamelachia pamelachia added the do-not-merge Not ready to be merged yet; pending other dependencies label Jul 31, 2025
@pamelachia pamelachia self-assigned this Jul 31, 2025
@vercel
Copy link

vercel bot commented Jul 31, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
studio-self-hosted ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 1, 2025 5:05am
studio-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 1, 2025 5:05am
6 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cms ⬜️ Ignored (Inspect) Aug 1, 2025 5:05am
studio ⬜️ Ignored (Inspect) Aug 1, 2025 5:05am
design-system ⬜️ Skipped (Inspect) Aug 1, 2025 5:05am
docs ⬜️ Skipped (Inspect) Aug 1, 2025 5:05am
ui-library ⬜️ Skipped (Inspect) Aug 1, 2025 5:05am
zone-www-dot-com ⬜️ Skipped (Inspect) Aug 1, 2025 5:05am

@supabase
Copy link

supabase bot commented Jul 31, 2025

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 ↗︎.

@coveralls
Copy link

coveralls commented Jul 31, 2025

Coverage Status

coverage: 83.039%. remained the same
when pulling fa832d8 on chore/remove-tos-banner
into 2fad79f on master.

Copy link
Member

@alaister alaister left a comment

Choose a reason for hiding this comment

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

LGTM!

  • Tested on preview

@pamelachia pamelachia removed the do-not-merge Not ready to be merged yet; pending other dependencies label Aug 1, 2025
@pamelachia pamelachia merged commit 9bdde50 into master Aug 1, 2025
19 of 20 checks passed
@pamelachia pamelachia deleted the chore/remove-tos-banner branch August 1, 2025 05:23
PLyn pushed a commit that referenced this pull request Aug 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants