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

Create GET API to fetch trips for the user #54 #83

Merged
merged 2 commits into from
Feb 14, 2025

Conversation

Suhanii-13
Copy link
Contributor

Changes made:

Updated the dashboard to display trips created by user .
Handled the case where no trips are found for a user.

Copy link

netlify bot commented Feb 11, 2025

Deploy Preview for friendly-crumble-3832b4 ready!

Name Link
🔨 Latest commit 97dc875
🔍 Latest deploy log https://app.netlify.com/sites/friendly-crumble-3832b4/deploys/67af64af3752020008de0a60
😎 Deploy Preview https://deploy-preview-83--friendly-crumble-3832b4.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

wonderlust/src/app/dashboard/page.js Outdated Show resolved Hide resolved
wonderlust/src/app/dashboard/page.js Outdated Show resolved Hide resolved
@Omeshcoding Omeshcoding added this to the Sprint 5 milestone Feb 11, 2025
) : (
<div className="grid gap-6 grid-cols-1 sm:grid-cols-2 lg:grid-cols-3">
{trips.map((trip, index) => (
<div
Copy link
Contributor

Choose a reason for hiding this comment

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

You can break down the dashboard page and create a new TripCard component here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sure i will create a separate PR for it

@GoldGroove06
Copy link
Contributor

image
the useState default values are rendered

@Suhanii-13 Suhanii-13 merged commit e776add into development Feb 14, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants