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

[feat] pull and display all plants #2

Conversation

kylezryr
Copy link
Contributor

What's new in this PR 🧑‍🌾

Description

  • Created Supabase client
  • Created Plant type based on schema
  • Created getPlantSeasonality query on Supabase and on TS

Screenshots

How to review

Next steps

Relevant links

Online sources

Related PRs

CC: @ccatherinetan

@ccatherinetan ccatherinetan force-pushed the kyleramachandran/tg-11-pull-and-display-plants-unstyled-from-plant_seasonality branch from eb382f1 to 923fbae Compare September 30, 2024 22:54
Co-authored-by: Kyle Ramachandran <[email protected]>
SashankBalusu pushed a commit that referenced this pull request Oct 9, 2024
…-from-plant_seasonality' of github.com:calblueprint/trap-garden into kyleramachandran/tg-11-pull-and-display-plants-unstyled-from-plant_seasonality
@kylezryr kylezryr marked this pull request as ready for review October 14, 2024 00:11
@@ -0,0 +1,24 @@
import React, { useEffect, useState } from 'react';
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this looks great! sorry for the change, but is it possible to create the seasonal-planting-guide page (route: app/seasonal-planting-guide) that uses the PlantList component?

Copy link
Collaborator

@ccatherinetan ccatherinetan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looking great kyle!

  1. my main request is to create the seasonal-planting-guide page to test out the PlantList component.
  2. lowk, we might change the schema a bit, so we might have to update that today during worksesh too. more soon!
  3. have you alr rebased? i'm not sure if there's a way to not add the createClient file again since it alr exists?
  4. let's rename this pr to "[feat] pull and display all plants"

@kylezryr kylezryr changed the title [chore] Create Supabase client [feat] pull and display all plants Oct 16, 2024
@ccatherinetan ccatherinetan merged commit 1811a59 into main Oct 21, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants