Skip to content

Pronina-todo-app-with-api#2264

Open
ProninaMariia wants to merge 3 commits into
mate-academy:masterfrom
ProninaMariia:develop
Open

Pronina-todo-app-with-api#2264
ProninaMariia wants to merge 3 commits into
mate-academy:masterfrom
ProninaMariia:develop

Conversation

@ProninaMariia

@ProninaMariia ProninaMariia commented Jul 3, 2026

Copy link
Copy Markdown

@brespect brespect 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 progress, but you DEMO LINK is not working

@ProninaMariia ProninaMariia requested review from brespect July 3, 2026 11:35

@Anton-Kuchmasov Anton-Kuchmasov 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, but using your personal USER_ID is cricital issue. Check out comment below

Comment thread src/App.tsx Outdated
import { TodoList } from './components/TodoList';
import { ErrorMessage } from './types/common';

const USER_ID = 1;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

You should retrieve your personal USER_ID from this page
https://mate-academy.github.io/react_student-registration/

And re-use it in your app

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

Well done

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