search

chingu-voyages / v46-tier3-team-37

Add-project-description-here | Voyage-46 | https://chingu.io/ | Twitter: https://twitter.com/ChinguCollabs
https://v46-tier3-team-37-rent-a-wrench.vercel.app/
0 stars 2 forks source link

Home page ui init #61

Closed SeanPaulson closed 10 months ago

SeanPaulson commented 10 months ago

Let me know if there is anything you guys think I should change. Also I was not sure if the location selection was text input or a button with a drop down menu. Let me know what yall think of the date pickers and if we just want images like in the wireframe or if the cards are ok.

SeanPaulson commented 10 months ago

Seeing that you have the server actions utilized in the API. My current PR will overwrite that.

As I understand this would not be the correct implementation of a server action. Instead we would want to have a given server action directly on/imported into our page.tsx's in the app folder. (as long as 'use client' isn't present)

Yes but the search results page is marked use client. Also I don't think it's necessarily a wrong way at the end of the day it's just a function that queries the database.