Skip to content

add task solution#2261

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

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

Conversation

@bodyafell

Copy link
Copy Markdown

DEMO LINK

I just did the last task, can you tell me how the previous ones were completed????

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

  1. after trying to delete several todos in a short amount of time, the loader only appears on the last todo pressed. (the same with changing status)

  2. loader does not appear after pressing the 'Toggle All' and 'Clear Completed' buttons

  3. split the app into several components (for example: header, main, footer errorNotification)

Comment thread src/App.tsx
setTodos(data);
})
.catch(() => {
showError('Unable to load todos');

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

create an enum value for errors

@bodyafell bodyafell requested a review from etojeDenys July 2, 2026 17:31

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

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