Skip to content

Conversation

@iirk1
Copy link

@iirk1 iirk1 commented Oct 30, 2025

Copy link

@FaiHamid FaiHamid left a comment

Choose a reason for hiding this comment

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

Good job!
Let`s just fix a few details:

src/App.tsx Outdated
Clear completed
</button>
</footer>
{/* Hide the footer if there are no todos */}

Choose a reason for hiding this comment

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

Remove unnecassery comments.

type ContextType = {
state: Todo[];
dispatch: React.Dispatch<Action>;
setFilter: (filter: 'all' | 'active' | 'completed') => void;

Choose a reason for hiding this comment

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

Create enum for filters.

@iirk1 iirk1 requested a review from FaiHamid October 31, 2025 16:19
Copy link

@vitalii-bondarenko-ua vitalii-bondarenko-ua left a comment

Choose a reason for hiding this comment

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

Looks good.

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