SupabaseClient returned when `.select()` is not included on query

[zlwaterfield]

Bug report

Describe the bug

When I do not include .select('*') on the query the SupabaseClient is returned and the request is not made. The documentation says: "If not used, all columns in the table will be returned." so both should work based on that wording.

With select

  async getAll() {
    const res = await SupabaseService
      .from<InvitationQuestion>('invitation_question')
      .select('*')
      .eq('wedding_id', this.weddingId)
    return res.body;
  }

res result:

Without select

  async getAll() {
    const res = await SupabaseService
      .from<InvitationQuestion>('invitation_question')
      .eq('wedding_id', this.weddingId)
    return res.body;
  }

res result:

• Version of supabase-js: v0.36.4

[kiwicopple]

I think this one should be a docs edit.

We don't won't "execute" the command until we know whether the users is selecting, updateing, deleteing etc

I'm guessing the "If not used, all columns in the table will be returned" is referring to not using the columns param. i.e.

await SupabaseService
      .from<InvitationQuestion>('invitation_question')
      .select('*')
      .eq('wedding_id', this.weddingId)
      .select()

The empty .select() should return all columns

[zlwaterfield]

Ah ok thanks, ya it might just be a document change needed. I can make a PR for that.

[zlwaterfield]

PR: https://github.com/supabase/supabase/pull/174

[zlwaterfield]

PR was merge.

[kiwicopple]

Thanks for the PR @zlwaterfield !