copied from my project into this directory

[mykhailonl]

Preview

[volodymyr-soltys97]

Hi, you need to add the demo link from readme.md file to the PR description

[mykhailonl]

DEMO LINK

[TarasHoliuk]

So you created a copy to add it to the portfolio? In this case, you shouldn't fork it from mate-academy repository (there is a special task for it on the platform). Also, the fork doesn't make sense since you actually use a completely new code. So commands (scripts) in package.json now doesn't work properly (they try to work with mate-academy scripts). So you have to fix this part :)

[mykhailonl]

So you created a copy to add it to the portfolio? In this case, you shouldn't fork it from mate-academy repository (there is a special task for it on the platform). Also, the fork doesn't make sense since you actually use a completely new code. So commands (scripts) in package.json now doesn't work properly (they try to work with mate-academy scripts). So you have to fix this part :)

I've asked what to do in the situation like that, when I've created a project from scratch, but still want to get a review from a mentors and get a task completed on a platform and got a reply, that I need to copy it to a fork.
What do I need to do in that case? :)

[TarasHoliuk]

1. Add cursor: pointer to all interactive elements:
   

2. The footer always should be at the bottom of the page:
   

Also, there is a bug:

• select 4 items/page
• select the last (or in the second part of pagination) page
• select 16 items/page
• observe no products displayed

Solution: set the first page when changing items/page.

3. This button leads to 404 page - fix it (make it anchor link or better just duplicate gitHub link):
   

4. Cursor pointer for buttons:
   

5. Empty Favorites (and Cart) page doesn't look good:
   

Make the message text bigger and (maybe) center it.

[TarasHoliuk]

I've asked what to do in the situation like that, when I've created a project from scratch, but still want to get a review from a mentors and get a task completed on a platform and got a reply, that I need to copy it to a fork.
What do I need to do in that case? :)

Got it now, thank you. Generally, you did everything right in this case) It would be nice to add such a comment in PR description at the very beginning so any mentor who's reviewing the task knows it. But thanks for the fast response 👍

So the previous comment is still relevant: delete redundant dependencies (mate-scripts for example). I see you've added eslint, so add commend to run lint. Same for other instruments if there is something else.

Usually, this task review mostly means testing your deployed solution. Add comment if you want more detailed code review 🙂 For now - fix current issues)

[TarasHoliuk]

Check comments

[mykhailonl]

Check comments

will fix everything tomorrow and will make a re-request, thanks!

[volodymyr-soltys97]

Good job 👍
To improve:

1. This block in the basket must be on the same line, fix it

2. Add hover effects here

[volodymyr-soltys97]

Almost done!
Sometimes this error occurs on Vercel, use better github pages for deployment demo link

[mykhailonl]

Almost done! Sometimes this error occurs on Vercel, use better github pages for deployment demo link

really struggling with deployment at pages, heres a last link to vercel, also you can click Production at my repo deployments

DEMO VERCEL
GITHUB REPO

[volodymyr-soltys97]

Hi, The link is currently not working

It should be possible to check the demo without authorization, since in the future companies will look at the demo, it should be done as conveniently as possible and without additional authorizations

If you need help with github pages deployment feel free in the fe_chat

[mykhailonl]

Had a lot of problems with deploying the page, but now all the urls / file paths are fixed, so should be available.
DEMO

[volodymyr-soltys97]

Great! 🔥