Skip to content

Solution#2405

Open
cassiaqueiroz wants to merge 1 commit into
mate-academy:masterfrom
cassiaqueiroz:develop
Open

Solution#2405
cassiaqueiroz wants to merge 1 commit into
mate-academy:masterfrom
cassiaqueiroz:develop

Conversation

@cassiaqueiroz

Copy link
Copy Markdown

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

That's fine. Reminder for next steps of the project: Don't leave the comments in the code.

Comment thread src/App.tsx
@@ -16,12 +55,15 @@ export const App: React.FC = () => {
<div className="todoapp__content">
<header className="todoapp__header">
{/* this button should have `active` class only if all todos are completed */}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

For next projects, you should not leave the comments in the code.

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