Skip to content

Complete task use context#1325

Open
RVDotsenko wants to merge 2 commits into
mate-academy:masterfrom
RVDotsenko:feat/use-react-context
Open

Complete task use context#1325
RVDotsenko wants to merge 2 commits into
mate-academy:masterfrom
RVDotsenko:feat/use-react-context

Conversation

@RVDotsenko

Copy link
Copy Markdown

@etojeDenys etojeDenys left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

good job, to improve:

  1. visual part looks weird
Image
  1. there is a problem after renaming the todo to a space. make sure to trim() the new todo text
Image

@RVDotsenko RVDotsenko requested a review from etojeDenys June 21, 2026 19:13

@etojeDenys etojeDenys left a comment

Copy link
Copy Markdown

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.

3 participants