greptileai / supabase-mirror / 23023063383 / 1

73%

master: 73%

push

github

fix: remove connectionString check (#43091)

## Problem
In local/self-hosted mode, tables incorrectly show "API disabled" even
when they have valid RLS policies.

This happens because:
1. `project.connectionString` is `null` for local/self-hosted (defaults
to null per API types)
2. `useTablesRolesAccessQuery` has `!!connectionString` in its enabled
condition
3. The query never runs, so the UI can't determine actual API access
status


## Solution
Removed `!!connectionString` check from `useTablesRolesAccessQuery`'s
enabled condition.

This is safe cause:
- likewise `executeSql` already handles null/empty connectionString via
`connectionString ?? ''`
For local/self-hosted, the query works without connectionString (pg-meta
uses direct database connection)
The query now runs and correctly works

Before:
<img width="1577" alt="Before fix - API disabled shown incorrectly"
src="https://github.com/user-attachments/assets/0510fe38-a4ff-4898-aacb-b2ec8f1a2182"
/>

After:
<img width="1128" alt="After fix - API status displays correctly"
src="https://github.com/user-attachments/assets/5784b76b-f7e7-4281-ac40-57408a17a294"
/>

- Closes #43081

Co-authored-by: Andrey A. <56412611+aantti@users.noreply.github.com>

835 of 946 branches covered (88.27%)

Branch coverage included in aggregate %.

3088 of 4413 relevant lines covered (69.98%)

110.63 hits per line