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

포트폴리오, 전체 코드 리팩토링 #142

Closed
6 tasks done
1jeongg opened this issue Nov 4, 2023 · 0 comments · Fixed by #145
Closed
6 tasks done

포트폴리오, 전체 코드 리팩토링 #142

1jeongg opened this issue Nov 4, 2023 · 0 comments · Fixed by #145
Assignees
@1jeongg
Labels
In-progress 진행 중 portfolio 포트폴리오 관련 기능 refactor

Comments

@1jeongg
Copy link
Member

1jeongg commented Nov 4, 2023 •
edited
Loading

📌 설명

포트폴리오 관련 로직 점검 및 메서드 분리 등의 리펙토링 작업

✅ 할 일

  • 포트폴리오 리팩토링
  • 포트폴리오 상세 조회시 이미지 없으면 Portfolio Not Found 에러 -> 이미지가 없어도 포트폴리오 조회는 가능해야하므로 로직 변경
  • Portfolio에 plannerName 필드 추가
  • portfolio/self 에 isLiked 필드 삭제
  • 로그인, 회원가입 시 responseBody에 userId 추가
  • user.getInfo() 로 되어있는 부분 user.getUser()로 변경
@1jeongg 1jeongg added In-progress 진행 중 test 테스트 코드 작성 portfolio 포트폴리오 관련 기능 refactor labels Nov 4, 2023
@1jeongg 1jeongg self-assigned this Nov 4, 2023
@1jeongg 1jeongg changed the title 포트폴리오 리팩토링 및 테스트 코드 작성 포트폴리오 리팩토링 Nov 4, 2023
@1jeongg 1jeongg changed the title 포트폴리오 리팩토링 포트폴리오, 전체 코드 리팩토링 Nov 4, 2023
@1jeongg 1jeongg mentioned this issue Nov 4, 2023
Merged
@1jeongg 1jeongg removed the test 테스트 코드 작성 label Nov 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@1jeongg 1jeongg

Labels
In-progress 진행 중 portfolio 포트폴리오 관련 기능 refactor
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

포트폴리오, 전체 코드 리팩토링 1jeongg/Team5_BE
1 participant
@1jeongg