Skip to content

app_with_api#2153

Open
7uckingbad wants to merge 2 commits intomate-academy:masterfrom
7uckingbad:develop
Open

app_with_api#2153
7uckingbad wants to merge 2 commits intomate-academy:masterfrom
7uckingbad:develop

Conversation

@7uckingbad
Copy link

No description provided.

@7uckingbad
Copy link
Author

Copy link

@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.

Well done!

// }}
// >
<input
// key="editing-input" // тоже тестик хз хуйня какая-то

Choose a reason for hiding this comment

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

Please try to avoid unnecessary inline comments and remove unprofessional ones like this

Copy link
Author

Choose a reason for hiding this comment

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

ок , а подскажите еще как лучше вот всю вот эту кашу в голову собрать в кучу , а то как-то после задания я вообще не понимаю как его делать

Choose a reason for hiding this comment

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

Nice job on this task — Bogdan also mentioned in the previous review that you handled it well. GJ!

Out of the three parts of the React Todo App, which one was the most challenging for you?

If some parts felt difficult, that's totally normal. The more you practice, write code, and debug your own solutions, the easier these problems will become over time. 🙂

Copy link
Author

Choose a reason for hiding this comment

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

да я просто понял что я путаюсь в каждой отдельной теме , от передачи пропсов до составления логики функций на добавление и т.д , я удмал может есть какой-то тренажер или , просто пробовать самому такое приложение написать без АПИ?

Choose a reason for hiding this comment

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

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