Skip to content

Conversation

dnywh
Copy link
Contributor

@dnywh dnywh commented Oct 2, 2025

What kind of change does this PR introduce?

Makes the Postgres upgrade experience clearer.

  • Fixes DEPR-80
  • Fixes DEPR-82

What is the current behavior?

  • The ObjectsToBeDroppedWarning just links to the docs. It doesn’t make it clear that the user needs to remove the affected function(s).
  • The Service Versions section doesn’t make it clear that this is where the user goes to upgrade Postgres
    • This is exacerbated by the fact that the API call to determine upgrade eligibility can take ~10 seconds

What is the new behavior?

Specifically for the ObjectsToBeDroppedWarning:

Before After
Infrastructure  Supabase Infrastructure  Supabase

To address upgrade clarity: the description for the Service Versions section has been amended.

Additional context

This is a follow up from #38904.

I would prefer that each line in ObjectsToBeDroppedWarning links out directly to its corresponding function. For example:

<Button size="tiny" type="default" asChild>
  <Link href={`/project/${ref}/database/functions?search=${obj}`}>Manage</Link>
</Button>

But the API only returns objects_to_be_dropped as an array of strings like:

  • function public.test_noop_plv8 is using obsolete language plv8

That requires backend changes to provide the name of the function and name of the extension, not just the string.

@dnywh dnywh requested a review from a team as a code owner October 2, 2025 05:28
@vercel
Copy link

vercel bot commented Oct 2, 2025

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

Project Deployment Preview Comments Updated (UTC)
studio-self-hosted Ready Ready Preview Comment Oct 5, 2025 11:49pm
studio-staging Ready Ready Preview Comment Oct 5, 2025 11:49pm
6 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
cms Ignored Ignored Oct 5, 2025 11:49pm
studio Ignored Ignored Oct 5, 2025 11:49pm
design-system Skipped Skipped Oct 5, 2025 11:49pm
docs Skipped Skipped Oct 5, 2025 11:49pm
ui-library Skipped Skipped Oct 5, 2025 11:49pm
zone-www-dot-com Skipped Skipped Oct 5, 2025 11:49pm

@supabase
Copy link

supabase bot commented Oct 2, 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 Oct 2, 2025

Coverage Status

coverage: 70.791%. remained the same
when pulling 51734c6 on dnywh/chore/improve-postgres-upgrade-experience
into 1ae6f4f on master.

…adeWarnings.tsx

Co-authored-by: Charis <26616127+charislam@users.noreply.github.com>
@dnywh dnywh requested a deployment to Studio E2E Tests October 5, 2025 23:44 — with GitHub Actions Waiting
@dnywh dnywh merged commit 311cc56 into master Oct 5, 2025
19 of 20 checks passed
@dnywh dnywh deleted the dnywh/chore/improve-postgres-upgrade-experience branch October 5, 2025 23:52
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.

3 participants