Skip to content

add task solution#987

Open
Lazarin123 wants to merge 2 commits into
mate-academy:masterfrom
Lazarin123:develop
Open

add task solution#987
Lazarin123 wants to merge 2 commits into
mate-academy:masterfrom
Lazarin123:develop

Conversation

@Lazarin123
Copy link
Copy Markdown

add task solution

Copy link
Copy Markdown

@IanLuan IanLuan left a comment

Choose a reason for hiding this comment

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

Hi Samuel, it's missing the DEMO LINK. Please run the deploy.

@Lazarin123
Copy link
Copy Markdown
Author

Não tem campo para add o DEMO LINK

@Lazarin123 Lazarin123 requested a review from IanLuan May 6, 2026 01:18
Copy link
Copy Markdown

@IanLuan IanLuan left a comment

Choose a reason for hiding this comment

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

I couldn’t run the project because some components are being imported incorrectly, such as ProductCard on the Favorites page.

Please review the imports and make sure all pages are running correctly.

@Lazarin123 Lazarin123 requested a review from IanLuan May 6, 2026 17:48
@Lazarin123
Copy link
Copy Markdown
Author

ajustado

Copy link
Copy Markdown

@IanLuan IanLuan left a comment

Choose a reason for hiding this comment

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

Great job!!

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.

2 participants