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

[PC-194] 이미지 등록 API #15

Open
wants to merge 6 commits into
base: develope
Choose a base branch
from

Conversation

Lujaec
Copy link
Member

@Lujaec Lujaec commented Dec 30, 2024

🔗 관련 이슈

PC-194

✨ 작업 내용

  • 파일 업로드 / 삭제를 위한 S3 서비스 클래스 개발
  • 프로필 이미지 업로드를 위한 서비스 클래스 개발
  • 프로필 이미지 업로드를 위한 컨트롤러 개발

✅ 체크리스트

  • 코드가 정상적으로 컴파일되나요?
  • 테스트 코드를 통과했나요?
  • merge할 브랜치의 위치를 확인했나요?
  • Label을 지정했나요?

🎃 새롭게 알게된 사항

📋 참고 사항

  • 코드 리뷰 시 논의가 필요한 사항이나 배포 관련 주의 사항을 추가합니다.
  • 버킷은 임시용으로 테스트하기 위해 회사에서 사용하고 있던 버킷을 사용했습니다,,

@Lujaec Lujaec requested a review from devchlee12 December 30, 2024 16:54
@Lujaec Lujaec self-assigned this Dec 30, 2024
Copy link
Member

@devchlee12 devchlee12 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!
사진 업로드 부분은 이후 presigned url 방식으로 바꾸는 것도 좋을 것 같습니다!

@PostMapping("/images")
@Operation(summary = "프로필 이미지 등록", description = "업로드한 이미지지를 버킷에 등록합니다.", tags = {"profileImage"})
@ApiResponse(responseCode = "200", description = "이미지가 버킷에 저장되었습니다.")
public ResponseEntity<CommonResponse<String>> checkNickname(@RequestParam("file") MultipartFile file) throws IOException {
Copy link
Member

Choose a reason for hiding this comment

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

혹시 프로필 이미지 등록 메서드 이름이 checkNickname인 이유가 무엇인가요?

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

Successfully merging this pull request may close these issues.

2 participants