Skip to content

Conversation

joshenlim
Copy link
Member

@joshenlim joshenlim commented Sep 25, 2025

Context

On the topic of enabling users to self debug issues, opting to provide contextual errors in the table editor especially when applying filters.

A common issue users run into is when providing a filter value that doesn't match the column data type, understandably with the current error UI it looks like it's something wrong with the dashboard. Opting to show actionable feedback to users when running into errors - we can incrementally expand this (We've already got one for foreign tables).

Before

image

Changes involved

  • Add error UI for "invalid input syntax" errors
    • Added a "remove filters" CTA as well to allow users to quickly reset back to a working state
    image
  • Add error UI for "could not identify ordering operator" errors
    • Added a "remove sorts" CTA as well to allow users to quickly reset back to a working state.
    image
  • Also opting to set retry to false when fetching table rows
    • I reckon we don't need to retry the network request if the request is failing, and just immediately show the error to users. Otherwise atm it looks like the dashboard is really slow as it retries 3 times before finally showing the error
  • Opt to not hide JSON columns from sort dropdown
    • We currently are, which can be confusing - opting to show a tooltip instead
      image

@joshenlim joshenlim requested a review from a team as a code owner September 25, 2025 09:40
Copy link

vercel bot commented Sep 25, 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 Sep 25, 2025 1:39pm
studio-staging Ready Ready Preview Comment Sep 25, 2025 1:39pm
6 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
cms Ignored Ignored Sep 25, 2025 1:39pm
studio Ignored Ignored Sep 25, 2025 1:39pm
design-system Skipped Skipped Sep 25, 2025 1:39pm
docs Skipped Skipped Sep 25, 2025 1:39pm
ui-library Skipped Skipped Sep 25, 2025 1:39pm
zone-www-dot-com Skipped Skipped Sep 25, 2025 1:39pm

Copy link

supabase bot commented Sep 25, 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 ↗︎.

throw new Error(
`Error fetching table rows: ${error instanceof Error ? error.message : 'Unknown error'}`
)
throw new Error(error instanceof Error ? error.message : 'Unknown error')
Copy link
Member Author

Choose a reason for hiding this comment

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

The prefix here should be handled on the component that's consuming the error

@coveralls
Copy link

coveralls commented Sep 25, 2025

Coverage Status

coverage: 69.37%. remained the same
when pulling 1fba6ad on chore/contextual-errors-for-table-editor
into 205a8bd on master.

Copy link
Contributor

github-actions bot commented Sep 25, 2025

Studio E2E Results

  • Total: 1
  • Passed: 1
  • Failed: 0
  • Skipped: 0
  • Timed out: 0
  • Interrupted: 0
  • Flaky: 0
  • Duration: 2.3s

Artifacts: https://github.com/supabase/supabase/actions/runs/18009239922

Last updated: Thursday 25, September, 2025 14:04:05 (UTC)

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

@joshenlim joshenlim merged commit b994566 into master Sep 30, 2025
20 checks passed
@joshenlim joshenlim deleted the chore/contextual-errors-for-table-editor branch September 30, 2025 01:34
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