Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support bulk update with array of partial items #325

Closed
aiibe opened this issue Sep 8, 2022 · 5 comments
Closed

Support bulk update with array of partial items #325

aiibe opened this issue Sep 8, 2022 · 5 comments
Labels
enhancement New feature or request postgrest-patch-needed Requires a change on PostgREST

Comments

@aiibe
Copy link

aiibe commented Sep 8, 2022

I am able to bulk update items (even without a filter) with an array of partial items.
Works fine but I had to cast my values .update(values as unknown as Partial<T>) while they are actually Partial<T>[].
Looking in the source code, it seems like update accept only a value at a time.

@soedirgo
Copy link
Member

soedirgo commented Sep 9, 2022

Does this work with @supabase/postgrest-js@rc or @supabase/supabase-js@rc? We'll be releasing these as new versions soon.

@aiibe
Copy link
Author

aiibe commented Sep 9, 2022

@soedirgo Haven't test with @supabase/postgrest-js@rc. I used @supabase/postgrest-js latest release.

@soedirgo
Copy link
Member

soedirgo commented Sep 27, 2022

Actually, rereading this again, does bulk update actually work for you right now? AFAIK this is still a WIP upstream: PostgREST/postgrest#1959

cc @steve-chavez

I should've looked more closely :) from the same issue: PostgREST/postgrest#1959 (comment)

So bulk update is actually not supported atm.

@soedirgo
Copy link
Member

soedirgo commented Sep 27, 2022

Thought we had an issue tracking this - sorry about that! I'll repurpose this issue to track the bulk update feature.

@soedirgo soedirgo changed the title Bulk update doc Support bulk update with array of partial items Sep 27, 2022
@soedirgo soedirgo added enhancement New feature or request postgrest-patch-needed Requires a change on PostgREST labels Sep 27, 2022
@soedirgo
Copy link
Member

Closing in favor of #174

@soedirgo soedirgo closed this as not planned Won't fix, can't repro, duplicate, stale May 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request postgrest-patch-needed Requires a change on PostgREST
Projects
None yet
Development

No branches or pull requests

2 participants