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

[PR] 게시물 기능 개발 #43

Merged
merged 17 commits into from
Apr 11, 2024
Merged

[PR] 게시물 기능 개발 #43

merged 17 commits into from
Apr 11, 2024

Conversation

jlee38266
Copy link
Contributor

@jlee38266 jlee38266 commented Apr 11, 2024

이 PR이 해결하는 문제

게시물의 카테고리별 게시글 목록 조회, 상세 조회, 작성, 수정, 삭제, 내가 작성한 게시글 목록 조회 기능 구현

변경 내용

  • 게시물에 필요한 기능을 추가합니다.

테스트

  • postman으로 api 요청 테스트 완료

관련 이슈

#25 #29 #24

jlee38266 added 16 commits April 5, 2024 16:38
나중에 사용할 예외처리를 위한 exceptions 디렉토리도 추가되었습니다.
디렉토리의 경로가 잘못되어 있어 수정합니다.
현재는 테스트 코드를 작성할 수 있는 기반 구축 및 진행중입니다.
# Conflicts:
#	OnionHotSayYo/build.gradle
#	OnionHotSayYo/src/main/java/org/omoknoone/onionhotsayyo/OnionHotSayYoApplication.java
현재 작업 중인 상태입니다. 계속해서 수정 될 예정입니다.
테스트 코드의 경우 modifyPost는 테스트 코드 확인을 보류합니다.
String -> LocalDateTime으로 번경 및 기능 구현 편의를 위해 다시 lombok 어노테이션 사용
@jlee38266 jlee38266 added the feature 새로운 기능을 추가했을 때 label Apr 11, 2024
@jlee38266 jlee38266 added this to the OnionHotSayYo-Server milestone Apr 11, 2024
@jlee38266 jlee38266 requested review from ms1011 and orlzlL April 11, 2024 06:16
@jlee38266 jlee38266 self-assigned this Apr 11, 2024
Copy link
Contributor

@ms1011 ms1011 left a comment

Choose a reason for hiding this comment

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

테스트코드까지 작성하면서 하기 힘들었을텐데 좋습니다

@orlzlL
Copy link
Contributor

orlzlL commented Apr 11, 2024

큰 기능 개발 고생 많으셨씁니다

@ms1011 ms1011 merged commit e2f41b8 into develop Apr 11, 2024
@jlee38266 jlee38266 deleted the feature/post branch April 11, 2024 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 새로운 기능을 추가했을 때
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants