Skip to content

For asking in a chat#974

Open
Natalia2408-forever wants to merge 8 commits into
mate-academy:masterfrom
Natalia2408-forever:develop
Open

For asking in a chat#974
Natalia2408-forever wants to merge 8 commits into
mate-academy:masterfrom
Natalia2408-forever:develop

Conversation

@Natalia2408-forever
Copy link
Copy Markdown

@Natalia2408-forever Natalia2408-forever commented Apr 18, 2026

Copy link
Copy Markdown

@Anton-Kuchmasov Anton-Kuchmasov left a comment

Choose a reason for hiding this comment

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

Task is not done yet, PR was opened for stage review

Left comments in fe_chat: https://mate.academy/chats/12392?selectedMessageId=11052609

Copy link
Copy Markdown

@brespect brespect left a comment

Choose a reason for hiding this comment

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

Good progress, but we need your DEMO LINK in Pull Request description to review the changes

@Natalia2408-forever
Copy link
Copy Markdown
Author

Copy link
Copy Markdown

@brespect brespect left a comment

Choose a reason for hiding this comment

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

Good progress, but I can't see your DEMO LINK in Pull Request description

@Natalia2408-forever
Copy link
Copy Markdown
Author

Copy link
Copy Markdown

@2pasha 2pasha left a comment

Choose a reason for hiding this comment

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

good job! 👏

here are some improvements:

  1. color picker doesn't work
Image
  1. user should have ability to visit product details page by clicking here
Image

Copy link
Copy Markdown

@2pasha 2pasha left a comment

Choose a reason for hiding this comment

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

lgtm 🔥

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.

4 participants