Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

독후감 리스트 반환 api #98

Merged
merged 9 commits into from
Apr 20, 2024
Merged

독후감 리스트 반환 api #98

merged 9 commits into from
Apr 20, 2024

Conversation

ta2ye0n
Copy link
Contributor

@ta2ye0n ta2ye0n commented Apr 20, 2024

💡 개요

독후감 리스트 반환 api를 구현했습니다

📃 작업내용

독후감 리스트 반환 api를 구현했습니다
본인이 아닐 경우 예외가 발생합니다

🔀 변경사항

🙋‍♂️ 질문사항

⚗️ 사용법

🎸 기타

@ta2ye0n ta2ye0n self-assigned this Apr 20, 2024

@Override
public BookListResponse toListDto(Book book) {
return BookListResponse.builder()
Copy link
Member

Choose a reason for hiding this comment

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

@Override 없애도 될거같아용

Copy link
Contributor Author

Choose a reason for hiding this comment

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

c3ed979

수정했습니다

private String plot;
private String date;
}

Copy link
Member

Choose a reason for hiding this comment

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

date를 String말구 LocalDate로 하는 건 어떤가요??
그러면 BookConverter에서 굳이 형식을 지정하지 않아도 되고 간결해질 거 같아요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

5ad0aa6

수정했습니다

.plot(book.getPlot())
.date(book.getCreateAt().format(DateTimeFormatter.ofPattern("yyyy-MM-dd")))
.build();
}
Copy link
Member

Choose a reason for hiding this comment

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

public BookListResponse toListDto(Book book) {
        return BookListResponse.builder()
                .id(book.getId())
                .title(book.getTitle())
                .plot(book.getPlot())
                .date(LocalDate.from(book.getCreateAt()))
                .build();
    }

이렇게 수정하면 될거같아요오

Copy link
Contributor Author

Choose a reason for hiding this comment

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

c0378e3

수정했습니다

Copy link
Member

@Umjiseung Umjiseung left a comment

Choose a reason for hiding this comment

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

수고하셧습니당

@ta2ye0n ta2ye0n merged commit be6d8a9 into develop Apr 20, 2024
1 check passed
@ta2ye0n ta2ye0n deleted the 92-get-book-report branch April 20, 2024 08:30
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