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

[ST-360] ⭐️ 서비스 내부 S3 이미지 업로드 연동 #31

Merged
merged 14 commits into from
Jul 23, 2023

Conversation

w8385
Copy link
Member

@w8385 w8385 commented Jul 23, 2023

업데이트 유형

  • Hot Fix
  • Release
  • Develop
  • Others

업데이트 개요

  • 사용자 프로필 및 문제 목록에서 사진을 확인하기 위함

업데이트 요약

  • User, Problem 객체 내부 imageURL 변수명 수정 > image
  • User 및 Request 생성 요청시 imageBase64, imageFormat 형식

해결한 문제

  • 사용자 생성시 프로필 이미지를 등록할 수 있습니다.
  • 과외 요청 생성시 문제 이미지를 등록할 수 있습니다.
  • 변수명을 의미가 명확하게 변경했습니다.
  • 함수 주석을 달아 코드 가독성을 높였습니다.

미해결 문제

  • 시뮬레이션시 이미지가 보이면 좋을 것 같습니다 (사심)

테스트

  • 유닛 테스트
  • 빌드 테스트 (통합 테스트)
  • 기타 유효성 테스트

수정 사항 진단

  • 코드가 이 프로젝트의 스타일 지침을 따릅니다.
  • 코드에 대한 자체 리뷰를 수행했습니다.
  • 변경 내역에 대해 주석을 작성했습니다.
  • 해당 PR에 대한 문서를 변경했습니다.
  • 기능, 수정 사항, 이슈에 대한 테스트 코드를 추가했습니다.
  • 변경 내용이 로컬 개발환경에서의 테스트를 통과했습니다.

@notion-workspace
Copy link

@w8385 w8385 force-pushed the feat/upload_image branch from 28828d4 to ae12da6 Compare July 23, 2023 02:06
@w8385 w8385 merged commit 8b6dda6 into dev Jul 23, 2023
@w8385 w8385 deleted the feat/upload_image branch July 23, 2023 04:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant