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

[#95] 페어 프로그래밍으로 코드 리팩터링 하기 #102

Merged
merged 35 commits into from
Feb 2, 2023

Conversation

JoosungKwon
Copy link
Member

@JoosungKwon JoosungKwon commented Feb 2, 2023

3차 스프린트

페어 프로그래밍을 통한 리팩터링 미션

  • 오프라인 스크럼 후 페어 프로그래밍 진행
  • 부족한 기능 추가
    • 사용자가 가진 시리즈 목록 조회 기능
    • 시리즈 업데이트 기능
    • 게시물 비공개 처리
    • Rest docs 수정
  • 일관성을 위한 코드 컨벤션 변경

연관된 이슈

close #95

JoosungKwon and others added 30 commits February 1, 2023 18:41
- create -> createSeries
- findByTitle -> findSeriesByTitle

Signed-off-by: kwonjoosung <[email protected]>
- 불필요한 코드 제거
- 규칙에 맞게금 네이밍 수정

Signed-off-by: kwonjoosung <[email protected]>
- URI 수정
- soft delete 추가
- update, delete 기능 추가
- 쿼리문 최적화
- 게시물 전체 조회: List<PostResponse> -> Page<PostResponse>
@JoosungKwon JoosungKwon added the ♻️ refactor 코드 리팩터링 label Feb 2, 2023
@JoosungKwon JoosungKwon added this to the Sprint - 3 milestone Feb 2, 2023
@JoosungKwon JoosungKwon self-assigned this Feb 2, 2023
@JoosungKwon JoosungKwon merged commit 29e343b into develop Feb 2, 2023
@JoosungKwon JoosungKwon deleted the refactor/95-code-merge branch February 2, 2023 05:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor 코드 리팩터링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

코드 일관성을 위한 리팩터링
3 participants