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

포트폴리오 READ, DELETE 로직 구현 #17

Closed
wants to merge 18 commits into from

Conversation

seokwns
Copy link
Contributor

@seokwns seokwns commented Sep 27, 2023

작업 내용

  • 포트폴리오 조회 로직 구현
  • 포트폴리오 삭제 로직 구현

이슈 번호

- 포트폴리오 가격을 위한 엔티티 생성 및 레포지토리 생성
- 포트폴리오 이미지를 위한 엔티티 생성 및 레포지토리 생성
- 가격 계산을 위한 객체 생성
- 포트폴리오 리스트 조회 구현
- 포트폴리오 상세 조회 구현
- 포트폴리오 삭제 구현
- EntityGraph 이름 변경
- DTO 변환 메서드 분리
- 포트폴리오 상세조회 시, Portfolio 정보를 조회가 아닌 PriceItem 을 join 해서 가져오도록 변경
- 포트폴리오 저장소에 대한 CRUD 성공 테스트 추가
- 포트폴리오 이미지를 불러올 때, Base64 인코딩된 이미지를 가져오도록 변경
- Base 64로 인코딩한 이미지로 변환하는 메서드 추출
- DTO 변환 로직 전체를 DTOConverter로 이동
- GetMapping 어노테이션에서 불필요한 값 삭제
- 포트폴리오 삭제 메서드 구현
@seokwns seokwns added enhancement New feature or request Done 진행 완료 portfolio 포트폴리오 관련 기능 In-progress 진행 중 and removed Done 진행 완료 labels Sep 27, 2023
seokwns and others added 10 commits September 27, 2023 18:02
# Conflicts:
#	sunsu-wedding/gradle/wrapper/gradle-wrapper.jar
- 테이블명 변경
- 테스트용 포트폴리오 데이터 추가
- 포트폴리오 조회 및 삭제 성공 테스트 추가
- 포트폴리오 조회, 삭제에 대한 실패 테스트 추가
- PR 충돌 해결
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request In-progress 진행 중 portfolio 포트폴리오 관련 기능
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants