Skip to content

[feat] 상품 이미지 수정 API#31

Merged
dtd1614 merged 2 commits intodevfrom
K5P-40/feat/crud-상품
Jul 9, 2024

Hidden character warning

The head ref may contain hidden characters: "K5P-40/feat/crud-\uc0c1\ud488"
Merged

[feat] 상품 이미지 수정 API#31
dtd1614 merged 2 commits intodevfrom
K5P-40/feat/crud-상품

Conversation

@Koneweekk
Copy link
Copy Markdown
Contributor

@Koneweekk Koneweekk commented Jul 9, 2024

관련 이슈

구현 기능

  • 상품 이미지 수정 컨트롤러 작성
  • 상품 이미지 수정 서비스 작성
  • 컨트롤러 테스트 코드 작성
  • 서비스 테스트 코드 작성
  • 중복되는 로직 함수화

참고 사항

Koneweekk added 2 commits July 9, 2024 09:15
- 상품 이미지 수정 컨트롤러 작성
- 상품 이미지 수정 서비스 작성
- 컨트롤러 테스트 코드 작성
- 서비스 테스트 코드 작성
- 중복되는 로직 함수화
- s3 디렉토리 경로 수정
Copy link
Copy Markdown
Member

@dtd1614 dtd1614 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jul 9, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@dtd1614 dtd1614 changed the title K5 p 40/feat/crud 상품 [feat] 상품 이미지 수정 API Jul 9, 2024
@dtd1614 dtd1614 added the feat 기능 개발 label Jul 9, 2024
@dtd1614 dtd1614 merged commit 69e2fde into dev Jul 9, 2024

if (multipartFile != null) {
uploadImage(item, multipartFile);
} else {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

early return 하여 else 문 삭제가 좋아 보이네요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat 기능 개발

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants