-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: queries to add and remove single or multiple trees #2
Conversation
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 think there is one small bug with addMultiple trees (see the other review comment), but other than that it looks good!
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.
Minor changes but wonderful job overall!
* supabase queries to add and remove trees. * query testing.
* supabase queries to add and remove trees. * query testing.
What's new in this PR
Description
Query support for invoking existing Supabase Database Functions for the following actions:
How to review
Examine
supabase/queries.ts
and use ofclient.ts
for Supabase queries. Check compliance with Linear sprint specs.Next steps
Testing
All queries and Supabase Functions are tested manually, through invoking buttons written to trigger queries in
App.tsx
. Adding single and multiple trees, and removing single and multiple trees (UUID JSON) worked as intended,and is reflected in the database.Relevant links
Online sources
N/A
Related PRs
N/A
CC: @christophertorres1