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.

Sign up for GitHub

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

Jump to bottom

Develop #550

Open
wants to merge 47 commits into
base: master
Choose a base branch
from
Open

Develop #550

wants to merge 47 commits into from

Conversation

siefimov
Copy link

@siefimov
add types
2acec7b
@siefimov
add app routes
199728c
@siefimov
add base styles
6ca1376
@siefimov
add state
7c07080
@siefimov
add enums
2900459
@siefimov
add api
86df324
@siefimov
add hooks
4516d36
@siefimov
add components
887a273
@siefimov
add pages
15e172c
@siefimov
add images and slides
63aaffa
@siefimov
update index.tsx
6f4303a
@siefimov
update Modal component and its styles
2a5dadb
@siefimov
update AddToCartButton component
9e66704
@siefimov
update not found page
d15ba6c
@siefimov
refactore product details page
db80f20
@siefimov
update product card
210f9d1
@siefimov
update footer
92722f1
@siefimov
update product slider
a1401c2
@siefimov
update new models component
e7e62a2
@siefimov
update product details page component
0ab066e
@siefimov
update products page component
f959e1b
@siefimov
update index.ts
1eb9198
@siefimov
add you may like component
26e2132
@siefimov
update loader styles
aa80a31
@siefimov
update common styles
5824102
@siefimov
fix extra spaces after last slider's item
8c1060a
@siefimov
add link to banner slide
753efb6
@siefimov
update app routes
bd983f5
@siefimov
add favicon
a9463cc
@siefimov
update app routes enum
df4f6eb
etojeDenys
etojeDenys suggested changes Dec 22, 2024
View reviewed changes
Copy link

@etojeDenys etojeDenys left a comment

Choose a reason for hiding this comment

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

image ask for some help how to fix that in the chat

@siefimov
Copy link
Author

I'm not sure I understand the issue.
The thing is, when I open the DEMO LINK, everything works correctly for me (see the screenshot below).
Could you provide me with more information about the circumstances under which this error occurs?

image

@siefimov siefimov requested a review from etojeDenys December 22, 2024 22:44
volodymyr-soltys97
volodymyr-soltys97 suggested changes Dec 23, 2024
View reviewed changes
Copy link

@volodymyr-soltys97 volodymyr-soltys97 left a comment

Choose a reason for hiding this comment

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

Hi, there is no content on the page, you need to fix it
image

VitaliyBondarenko1982
VitaliyBondarenko1982 approved these changes Dec 24, 2024
View reviewed changes
Copy link

@VitaliyBondarenko1982 VitaliyBondarenko1982 left a comment

Choose a reason for hiding this comment

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

Looks good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@VitaliyBondarenko1982 VitaliyBondarenko1982 VitaliyBondarenko1982 approved these changes

@etojeDenys etojeDenys Awaiting requested review from etojeDenys

@volodymyr-soltys97 volodymyr-soltys97 Awaiting requested review from volodymyr-soltys97

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@siefimov @VitaliyBondarenko1982 @volodymyr-soltys97 @etojeDenys