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
fix(supabase): change default value of meta.select to "*" #5841
Conversation
🦋 Changeset detectedLatest commit: 9675681 The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @issa012 thanks for catching up on this one! Added a comment about mocks. Other than that, LGTM.
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 9675681. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this CI Pipeline Execution
✅ Successfully ran 1 targetSent with 💌 from NxCloud. |
❌ Deploy Preview for refine-video-club failed.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I sent a commit to update the tests with respect to the changes in this PR and our supabase database. Thank you for the contribution @issa012!
PR Checklist
Please check if your PR fulfills the following requirements:
Bugs / Features
What is the current behavior?
If meta.select is not specified, update, create, updateMany, createMany methods will not return data.
What is the new behavior?
default value of meta.select is now "*", which selects all data to be returned
fixes #5679
Notes for reviewers