Skip to content

Conversation

@joshenlim
Copy link
Member

@joshenlim joshenlim commented Sep 29, 2025

Context

Edge case - but the Table Editor will error out when fetching rows for tables with names that are reserved keywords (e.g system_user) Fix is to wrap the table names in double quotes in the query from the query builder.

To test

  • Create a table with a name like "system_user" in the table editor
  • Table should not error out when loading rows after opening it

@vercel
Copy link

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

@github-actions
Copy link
Contributor

github-actions bot commented Sep 29, 2025

Studio E2E Results

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

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

Last updated: Monday 29, September, 2025 09:20:00 (UTC)

@joshenlim joshenlim marked this pull request as draft September 29, 2025 04:19
@joshenlim joshenlim closed this Sep 29, 2025
@joshenlim joshenlim reopened this Sep 29, 2025
@supabase
Copy link

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

@joshenlim joshenlim marked this pull request as ready for review September 29, 2025 08:45
@joshenlim joshenlim requested a review from a team as a code owner September 29, 2025 08:45
@coveralls
Copy link

Coverage Status

coverage: 69.874%. remained the same
when pulling 5ffbf34 on chore/fix-query-builder-wrap-table-schema-names-in-double-quotes
into 1973615 on master.

Copy link
Contributor

@charislam charislam left a comment

Choose a reason for hiding this comment

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

tested in preview

@joshenlim joshenlim merged commit f370cc6 into master Sep 30, 2025
19 checks passed
@joshenlim joshenlim deleted the chore/fix-query-builder-wrap-table-schema-names-in-double-quotes branch September 30, 2025 03:42
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