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: profile page skeleton #87

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MaheshtheDev
Copy link
Contributor

@MaheshtheDev MaheshtheDev commented Jun 29, 2024

Summary

Implement a feature to display the user's profile page skeleton and update the header component to include the user's profile link.

Details

  • The feat: profile page skeleton pull request aims to introduce a feature that displays the user's profile page skeleton and update the header component to include the user's profile link.
  • This pull request includes the following changes:
    • Removal of the AddIcon import from the @repo/ui/icons package in the header.tsx file.
    • Addition of a userId prop to the Header functional component in the header.tsx file.
    • Modification of the Header component to include a Link element that navigates to the user's profile page.
    • Addition of a page.tsx file under the apps/web/app/(dash)/user/[userId] folder, which defines a functional component for rendering the user's profile page skeleton.
    • Creation of a new getUserData function in the fetchers.ts file to fetch the logged-in user's data.
  • Refactoring details:
    • The Layout functional component in the layout.tsx file is updated to pass the userId prop to the Header component.
    • The Link element in the header.tsx file is updated to construct the user's profile URL dynamically using the userId prop.
    • The updated getUserData function in the fetchers.ts file is designed to fetch the logged-in user's data using the Drizzle ORM query interface. It returns an object containing the fetched user data or an error message, along with a success status indicator.

✨ Generated with love by Kaizen ❤️

Original Description image

Copy link

kaizen-bot bot commented Jun 29, 2024

Code Review

All Clear: This PR is ready to merge! 👍

✨ Generated with love by Kaizen ❤️

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.

None yet

1 participant