Skip to content

add a task solution#1781

Open
Mykyta-snacj wants to merge 1 commit intomate-academy:masterfrom
Mykyta-snacj:develop
Open

add a task solution#1781
Mykyta-snacj wants to merge 1 commit intomate-academy:masterfrom
Mykyta-snacj:develop

Conversation

@Mykyta-snacj
Copy link

No description provided.

Copy link

@mateacademy-ai-mentor mateacademy-ai-mentor left a comment

Choose a reason for hiding this comment

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

Review Summary

Great job! 🎉 Your implementation meets all the core requirements and works as expected. The only suggestion is to update the jsx option in your tsconfig.json to react-jsx (or react-jsxdev for development) if you're using React 17 or newer, as the current react value is deprecated in recent TypeScript versions. This is a minor configuration detail and doesn't affect your code's functionality. Keep up the good work! 🚀


✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨

Footnotes

  1. Rate AI review example

"src"
],
"compilerOptions": {
"jsx": "react",

Choose a reason for hiding this comment

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

The jsx option should be set to react-jsx (or react-jsxdev for development) for React 17+ projects. Using react is deprecated in newer TypeScript versions and may not work as expected. Please update this value if you are using a recent version of React and TypeScript.

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