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.

Sign up for GitHub

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

Jump to bottom

[BE] 핀(Pin) 패키지 코드 리팩터링 #151

Closed
10 tasks done
yoondgu opened this issue Aug 1, 2023 · 0 comments
Closed
10 tasks done

[BE] 핀(Pin) 패키지 코드 리팩터링 #151

yoondgu opened this issue Aug 1, 2023 · 0 comments
Assignees
@yoondgu
Labels
BE 백엔드 관련 이슈 refactor 리팩토링 관련 이슈 우선순위 : 중

Comments

@yoondgu
Copy link
Collaborator

yoondgu commented Aug 1, 2023 •
edited
Loading

🛠️ 작업 대상

핀 패키지(pin)에 포함된 모든 코드

관련 이슈 #111

✅ 작업 내용

  • 위치 중복 처리 로직 변경 (Location 리팩터링과 싱크 맞추기)
  • 코드 컨벤션 통일
  • 권한 확인 로직 수정 (fix)
  • RestDocs 테스트 작성
  • Pin, PinImage 생성 시 메서드 시그니처 수정 (PinInfo, Image 관련)
  • 그 외 테스트 보완 #212
  • 핀 이미지 수정 적용 #213
    • 핀 CRUD 명세에서 images 삭제하고, 핀 이미지 추가, 삭제, 조회 명세 만들기
  • 핀, 핀 이미지 연관 관계 매핑 관련 검토
  • 핀 이미지 관련 테스트 추가

📎 참고 자료

No response

⏰ 추정 시간

비관적 추정: 추가 리팩터링 항목이 생길 경우 10시간+a
낙관적 추정: 전체 6시간
@yoondgu yoondgu added BE 백엔드 관련 이슈 refactor 리팩토링 관련 이슈 labels Aug 1, 2023
@yoondgu yoondgu self-assigned this Aug 1, 2023
@yoondgu yoondgu moved this to In Progress in 2023-map-befine Aug 2, 2023
@yoondgu yoondgu mentioned this issue Aug 7, 2023
Merged
@yoondgu yoondgu closed this as completed Aug 9, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Done in 2023-map-befine Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yoondgu yoondgu

Labels
BE 백엔드 관련 이슈 refactor 리팩토링 관련 이슈 우선순위 : 중
Projects
2023-map-befine
Status: Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@yoondgu