Skip to content

Практическое задание 1 # Spring ToDo#39

Open
GeorgyVShefer wants to merge 17 commits into
EffectiveMobile:mainfrom
GeorgyVShefer:dev
Open

Практическое задание 1 # Spring ToDo#39
GeorgyVShefer wants to merge 17 commits into
EffectiveMobile:mainfrom
GeorgyVShefer:dev

Conversation

@GeorgyVShefer

Copy link
Copy Markdown

No description provided.

@@ -0,0 +1,26 @@
CREATE TABLE IF NOT EXISTS todo (

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Обычно мастер файл вбирает в себя другие файлы, сделай также, вынеси в отдельный файл

private final ToDoService toDoService;

@Override
public List<ToDoDto> getAll(int limit, int offset) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Необходимо сделать пагинацию для списка дел.
Создай DTO чтобы понимать на какой странице и куда идти дальше

import com.emobile.springtodo.dto.ToDoDto;
import com.emobile.springtodo.service.ToDoService;
import lombok.RequiredArgsConstructor;
import org.springframework.http.ResponseEntity;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Лишний импорт, форматирование поправь и убери лишние пробелы и отсупы

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