[SSF-22] Create temporary site index on homepage#41
Conversation
dburkhart07
left a comment
dburkhart07
left a comment
There was a problem hiding this comment.
small styling adjustment but aside from that looks great!
| </div> | ||
| </div> | ||
| ); | ||
| }; |
There was a problem hiding this comment.
This all looks really good visually. For consistency purposes, as we are using Chakra UI for this project, can we convert all of this to that? You can reference other files to see how we structure some pieces, but the headers can be replaced with Heading, sections with Box, lists and list items with List and ListItems, paragraph with Text. Can reference this here for how to implement all of these: https://chakra-ui.com/docs/components/concepts/overview
dburkhart07
left a comment
dburkhart07
left a comment
There was a problem hiding this comment.
LGTM!! I'd hold off on merging till Sam can take a look.
sam-schu
left a comment
sam-schu
left a comment
There was a problem hiding this comment.
"All Pantries" should be moved to the admin view section, and "Donation Management" to the food manufacturer view section. Also, could you please merge in changes from main and add Justin's new /volunteer-management page to the admin view section? Thanks!
…H/SSF-22-temporary-site-homepage
|
Just fixed it, thanks! @sam-schu |
…-homepage [SSF-22] Create temporary site index on homepage
4 participants
ℹ️ Issue
Closes 22
📝 Description
I added a temporary site homepage, which containers all of the pages in our application organized by different views (Pantry View, Food Manufacturer View, Admin View, etc.) The pages in our site aren't currently linked, so this homepage is meant to make testing the site more convenient, preventing users from having to type in the url manually.
✔️ Verification
I ran the homepage locally and tested each link to make sure it works. For the Pantry Dashboard and the Request Form, clicking on the links uses a value of 1.
🏕️ (Optional) Future Work / Notes
I had to change app.tsx to have an index route to the homepage because the root is displayed on every page. This prevents the index from appearing on every page.