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

Solution by Marco Yue #5

Open
wants to merge 23 commits into
base: main
Choose a base branch
from
Open

Conversation

yuekalong
Copy link

Hello I am Marco.
Here is my solution for the Candidate Assessment Test.

To start the project:

yarn install
yarn dev

Project Includes:

  • Home Page
    • Greeting Banner
    • Product Keyword Highlights
    • Product Description
    • Order Form
  • Privacy Page
  • Order Detail Page

Libraries I have used

  • Next.js
  • Lenis
  • Motion (Framer Motion)
  • Zustand
  • react-hook-form
  • Zod
  • Prisma
  • Sqlite3

And images are from PNGTree and Unsplash.

Something I would like to improve

  • Implement Page Transitions:
    Add smooth transitions between pages to enhance user experience (UX) across the project.

  • Enhance SEO:
    Optimize the HTML tags used throughout the site and incorporate relevant meta tags for each individual page to improve search engine visibility.

  • Organize Components:
    Structure the page components systematically and utilize Storybook for testing and documentation to ensure consistency.

  • Expand Testing Coverage:
    Develop additional test scripts to verify that all features of the project function correctly and efficiently, ensuring a smooth user experience.

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.

1 participant