-
Notifications
You must be signed in to change notification settings - Fork 1
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
[feat] pull and display all plants #2
Conversation
eb382f1
to
923fbae
Compare
Co-authored-by: Kyle Ramachandran <[email protected]>
…-from-plant_seasonality' of github.com:calblueprint/trap-garden into kyleramachandran/tg-11-pull-and-display-plants-unstyled-from-plant_seasonality
@@ -0,0 +1,24 @@ | |||
import React, { useEffect, useState } from 'react'; |
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.
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?
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.
looking great kyle!
- my main request is to create the
seasonal-planting-guide
page to test out thePlantList
component. - lowk, we might change the schema a bit, so we might have to update that today during worksesh too. more soon!
- have you alr rebased? i'm not sure if there's a way to not add the createClient file again since it alr exists?
- let's rename this pr to "[feat] pull and display all plants"
…-from-plant_seasonality' of github.com:calblueprint/trap-garden into kyleramachandran/tg-11-pull-and-display-plants-unstyled-from-plant_seasonality
What's new in this PR 🧑🌾
Description
Screenshots
How to review
Next steps
Relevant links
Online sources
Related PRs
CC: @ccatherinetan