feat: s3 이미지 업로드 기능 추가 #464
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kevstevie
approved these changes
Oct 4, 2023
| .build(), | ||
| RequestBody.fromBytes(image.getBytes()) | ||
| ); | ||
| return CLOUD_FRONT_DOMAIN_NAME + "/" + key; |
There was a problem hiding this comment.
p3:이거 지역변수나 stringformat 정적변수로 추출하는거 어떤가요
Comment on lines
+23
to
+26
| public ResponseEntity<Void> uploadImage(@RequestPart MultipartFile image) { | ||
| return ResponseEntity.created(URI.create(imageResourceService.uploadImage(image))) | ||
| .build(); | ||
| } |
There was a problem hiding this comment.
p3: service 반환값 로컬변수로 빼고 build 한줄에 쓰는게 더 이쁠 거 같아요
|
추가 코멘트하고 싶은게 생각났는데 service에서 uploader 같은 인터페이스를 의존하고 구현체로 s3 client 사용하는 객체를 만드는 것도 좋을 것 같슴다 |
CFalws
reviewed
Oct 5, 2023
| .region(Region.of(region)) | ||
| .build(); | ||
| } | ||
|
|
There was a problem hiding this comment.
이미지 업로드 실패시 어떻게 처리할지가 머리가 아픈 부분인 거 같네요~
이건 나중에 한 번 상의해보시죠!
| } | ||
| } | ||
|
|
||
| private String createKey(ImageExtension extension) { |
There was a problem hiding this comment.
url으로 이미지 직접 접근하려면 확장자가 필요합니다
| } | ||
|
|
||
| @PostMapping | ||
| public ResponseEntity<Void> uploadImage(@RequestPart MultipartFile image) { |
There was a problem hiding this comment.
저희 혹시 여러 이미지를 한 요청에 한 번에 보낼 수 있나요?
보낼 수 없다면 여러번 요청을 보내게 되는데 한 번 생각해볼 부분이 있을 것 같네요!
CFalws
approved these changes
Oct 5, 2023
poi1649
force-pushed
the
BE/feature/463-image-upload
branch
from
08c1474
to
ca41d12
Compare
poi1649
added a commit
that referenced
this pull request
Oct 19, 2023
* refactor: 이벤트 제거 (#384) * feat: 회차 멤버들의 투두 진행률 조회 api (#382) * feat: 회차 진행률 계산 구현 * feat: CloudWatch Actuator 연동 * feat: CloudWatch Actuator tag 수정 * fix: 클라우드 와치 메트릭을 production 환경에서만 수집하게 변경 * chore: 프로메테우스 의존성 제거 (#392) * feat: 스터디장이 탈퇴할 시 모집중인 스터디는 삭제하는 기능 구현 (#335) * feat: 스터디장이 탈퇴할 시 모집중인 스터디는 삭제하는 기능 구현 * refactor: on delete 추가 * chore: flyway v7 * fix: conflict 해소 * test: scenario 위치 수정 * refactor: 이름 수정 및 flyway 쿼리 수정 * refactor: 트랜잭션 통합 * test: sharedcontext 기능 추가 (#387) * test: sharedcontext 기능 추가 * refactor: 잔여 부분 수정 --------- Co-authored-by: Jae Min Yu <[email protected]> * 토큰 만료 확인 기능 (#405) * feat: 토큰 만료 확인 기능 구현 * fix: 테스트 변경 사항에 따른 버그 수정 * feat: 비회원 스터디 상세정보 조회 기능 구현 (#414) * feat: 비회원 스터디 상세정보 조회 기능 구현 - 스터디 상세 조회 API에서 역할을 제공하지 않도록 수정 - 스터디 멤버 역할 조회 API 구현 * feat: 스터디멤버 역할 조회 API 스웨거 적용 * test: 테스트 코드 주석 제거 * refactor: Pattern 객체 상수화 * style: 개행 추가 * test: assertAll 사용하도록 수정 * QA 버전 배포 (#413) * chore: 프로메테우스 의존성 제거 (#392) * feat: 스터디장이 탈퇴할 시 모집중인 스터디는 삭제하는 기능 구현 (#335) * feat: 스터디장이 탈퇴할 시 모집중인 스터디는 삭제하는 기능 구현 * refactor: on delete 추가 * chore: flyway v7 * fix: conflict 해소 * test: scenario 위치 수정 * refactor: 이름 수정 및 flyway 쿼리 수정 * refactor: 트랜잭션 통합 * test: sharedcontext 기능 추가 (#387) * test: sharedcontext 기능 추가 * refactor: 잔여 부분 수정 --------- Co-authored-by: Jae Min Yu <[email protected]> * 토큰 만료 확인 기능 (#405) * feat: 토큰 만료 확인 기능 구현 * fix: 테스트 변경 사항에 따른 버그 수정 * feat: 비회원 스터디 상세정보 조회 기능 구현 (#414) * feat: 비회원 스터디 상세정보 조회 기능 구현 - 스터디 상세 조회 API에서 역할을 제공하지 않도록 수정 - 스터디 멤버 역할 조회 API 구현 * feat: 스터디멤버 역할 조회 API 스웨거 적용 * test: 테스트 코드 주석 제거 * refactor: Pattern 객체 상수화 * style: 개행 추가 * test: assertAll 사용하도록 수정 --------- Co-authored-by: kevstevie <[email protected]> Co-authored-by: Jae Min Yu <[email protected]> * fix: flyway 수정 (#415) * Revert "QA 버전 배포 (#413)" (#416) This reverts commit 1bb1174. * 티어 계산 및 경험치 계산 로직 구현 (#397) * feat: 티어 계산 및 경험치 계산 로직 구현 * remove: study에서 최대 보상 계산하는 로직 제거 * refactor: 멤버가 성공적으로 진행한 라운드를 계산하는 기능을 스터디에게 부여 * fix: 없어진 괄호 복구 * test: 깨지는 테스트 수정 --------- Co-authored-by: kevstevie <[email protected]> * 피드 기능을 구현한다. (#404) * test: 피드 조회 기능 테스트 추가 * feat: 피드 조회 기능 추가 * feat: 일반 피드 작성 기능 추가 * feat: 피드 작성 / 조회 기능 추가 * fix: 배열로 시간 직렬화되는 버그 해결 * fix: 인증 및 피드 기능 분리 * Update backend/src/test/java/com/yigongil/backend/acceptance/steps/FeedSteps.java Co-authored-by: Jae Min Yu <[email protected]> * refactor: study service에서 피드를 찾도록 수정 * refactor: pagination에서 cursor를 이용하도록 변경 * fix: flyway추가 --------- Co-authored-by: Jae Min Yu <[email protected]> * refactor: 티어 -> 경험치 flyway 쿼리 추가 (#428) * refactor: 홈뷰 수정 (#430) * refactor: 홈뷰 수정 * refactor: 매직 넘버 수정 * fix: flyway 언더바 수정 * fix: flyway 타입 수정 * refactor: 선택 투두 삭제 (#432) * refactor: 선택 투두 삭제 * refactor: 투두 내용만 수정하게 변경 및 테스트 수정 * refactor: 피드 조회시에 스터디 이름도 반환 (#453) * refactor: 피드 조회시에 스터디 이름도 반환 * Revert "refactor: 피드 조회시에 스터디 이름도 반환" This reverts commit 34b4907. * refactor: conflict 해소 * refactor: aop로 exception handler에서 slack 의존성 제거 (#442) * refactor: aop로 exception handler에서 slack 의존성 제거 * refactor: slack 중복알림 해소 * feat: StudyV1, StudyV2 분리 * feat: Study 엔티티 수정 * fix: 스터디 엔티티 필드 추가 * refactor: PeriodUnit 제거 * 엔티티를 수정하며 발생한 컴파일 에러 해결 (#470) * fix: totalRoundCount, periodOfRound 삭제에 따른 컴파일 에러 해결 * fix: 엔티티 수정에 따른 컴파일 에러 임시 해결 * refactor: swagger 설정 변경 (#452) * refactor: swagger 설정 변경 * refactor: swagger url 설정 파일로 관리 * feat: 동적 쿼리를 통해 스터디 목록 필터링/정렬 (#475) * feat: 동적 쿼리를 통해 스터디 목록 필터링/정렬 * refactor: config v1 prefix 제거 * refactor: 패키지 수정 * refactor: v1 prefix 제거 * refactor: status null 처리 * refactor: response 이름 수정 * refactor: param을 enum으로 컨버팅 * refactor: enum 순서 수정 * refactor: 메서드 추출 * fix: 스터디 시작 할 때 요일을 선택하도록 수정 (#473) * feat: 스터디 개설 기능 구현 #468 * refactor: Study.startAt 제거 및 minimumWeeks와 meetingDaysCountPerWeek 추가 * fix: 스터디 목록 조회 API url 수정 * fix: 실패하는 스터디 일부 복구 및 주석 작성 * test: 실패하는 테스트 복구 * 스터디 상세조회 구현 & 멤버 인증 정보 조회 api 수정 (#484) * refactor: 상세 조회 수정 * feat: 멤버 인증 정보 조회 응답 수정 * feat: 스터디 종료 기능 추가 (#483) * fix: 과거 청산 및 새로운 시작 drop and restart (#490) * feat: 지원한 스터디 목록 조회 기능 (#485) * feat: 지원한 스터디 목록 조회 기능 * refactor: querydsl로 마이그래이션 * chore: 개행 * chore: 정렬 수정 * fix: conflict 해결 중 오류 수정 (#452) * feat: 주차별 회차 조회 기능 추가 (#494) * feat: 주차별 회차 조회 기능 추가 * refactor: 투두 -> 머스트두 변경, 생성 / 수정 통합 * Update backend/src/test/java/com/yigongil/backend/domain/round/RoundTest.java * Update backend/src/test/java/com/yigongil/backend/domain/round/RoundTest.java * fix: 마스터 탈퇴 시 정상적으로 스터디 삭제되게 수정 (#492) * fix: flyway v12 이름 수정 * fix: flyway v12 ddl 수정 * fix: flyway v12 ddl 수정 * fix: swagger 설정 변경 * fix: swagger 설정 변경 * fix: swagger redirect * Revert "fix: swagger redirect" This reverts commit a1cfaa8. * fix: swagger disable efault url * fix: swagger 설정 변경 * fix: flyway v12 이름 수정 * fix: flyway v12 ddl 수정 * fix: flyway v12 ddl 수정 * fix: swagger 설정 변경 * fix: swagger 설정 변경 * fix: swagger redirect * Revert "fix: swagger redirect" This reverts commit a1cfaa8. * fix: swagger disable efault url * fix: swagger 설정 변경 * feat: graceful shutdown (#510) Co-authored-by: Jae Min Yu <[email protected]> * chore: datadog 의존성 추가 * feat: s3 이미지 업로드 기능 추가 (#464) * feat: s3 이미지 업로드 기능 추가 * fix: 이미지 업로드 관련 빈이 로컬에서 로드되지 않도록 변경 * fix: url 수정 * fix: 스터디 멤버 삭제 로직을 study 안으로 이관 (#528) * fix: s3 directory 설정 수정 (#521) * feat: s3 이미지 업로드 기능 추가 * fix: 이미지 업로드 관련 빈이 로컬에서 로드되지 않도록 변경 * fix: url 수정 * fix: s3 directory 수정 * feat: 스터디 탈퇴 기능 (#526) Co-authored-by: Bojun Kim <[email protected]> * feat: 대기중인 스터디 조회 기능 구현 (#531) Co-authored-by: Bojun Kim <[email protected]> * refactor: 인증 조회 방식 수정 (#520) * refactor: 인증 조회 방식 수정 * refactor: optional로 수정 --------- Co-authored-by: Bojun Kim <[email protected]> * fix: collection에서 원소를 삭제하지 않도록 수정 (#537) * hotfix 배포 (#540) * fix: collection에서 원소를 삭제하지 않도록 수정 (#537) * hotfix: 이미지 컨트롤러 url 매핑 수정 (#539) * fix: collection에서 원소를 삭제하지 않도록 수정 * fix: ImageResourceController url매핑 수정 --------- Co-authored-by: Bojun Kim <[email protected]> * qa 버그 수정 버전 배포 (#548) * fix: collection에서 원소를 삭제하지 않도록 수정 (#537) * hotfix: 이미지 컨트롤러 url 매핑 수정 (#539) * fix: collection에서 원소를 삭제하지 않도록 수정 * fix: ImageResourceController url매핑 수정 * fix : 홈에서 해당 스터디의 누적잔디 조회 가능 (#544) * refactor: 누적 잔디 조회 기능 수정 * chore: todo 제거 * fix: 클라이언트 반환 이미지 경로 수정 (#546) Co-authored-by: Bojun Kim <[email protected]> --------- Co-authored-by: kevstevie <[email protected]> Co-authored-by: Jae Min Yu <[email protected]> --------- Co-authored-by: kevstevie <[email protected]> Co-authored-by: yujamint <[email protected]> Co-authored-by: kevstevie <[email protected]> Co-authored-by: kim daehee <[email protected]> Co-authored-by: kim dae hee <[email protected]>
yujamint
added a commit
that referenced
this pull request
Nov 16, 2023
* refactor: 이벤트 제거 (#384) * feat: 회차 멤버들의 투두 진행률 조회 api (#382) * feat: 회차 진행률 계산 구현 * feat: CloudWatch Actuator 연동 * feat: CloudWatch Actuator tag 수정 * fix: 클라우드 와치 메트릭을 production 환경에서만 수집하게 변경 * chore: 프로메테우스 의존성 제거 (#392) * feat: 스터디장이 탈퇴할 시 모집중인 스터디는 삭제하는 기능 구현 (#335) * feat: 스터디장이 탈퇴할 시 모집중인 스터디는 삭제하는 기능 구현 * refactor: on delete 추가 * chore: flyway v7 * fix: conflict 해소 * test: scenario 위치 수정 * refactor: 이름 수정 및 flyway 쿼리 수정 * refactor: 트랜잭션 통합 * test: sharedcontext 기능 추가 (#387) * test: sharedcontext 기능 추가 * refactor: 잔여 부분 수정 --------- Co-authored-by: Jae Min Yu <[email protected]> * 토큰 만료 확인 기능 (#405) * feat: 토큰 만료 확인 기능 구현 * fix: 테스트 변경 사항에 따른 버그 수정 * feat: 비회원 스터디 상세정보 조회 기능 구현 (#414) * feat: 비회원 스터디 상세정보 조회 기능 구현 - 스터디 상세 조회 API에서 역할을 제공하지 않도록 수정 - 스터디 멤버 역할 조회 API 구현 * feat: 스터디멤버 역할 조회 API 스웨거 적용 * test: 테스트 코드 주석 제거 * refactor: Pattern 객체 상수화 * style: 개행 추가 * test: assertAll 사용하도록 수정 * QA 버전 배포 (#413) * chore: 프로메테우스 의존성 제거 (#392) * feat: 스터디장이 탈퇴할 시 모집중인 스터디는 삭제하는 기능 구현 (#335) * feat: 스터디장이 탈퇴할 시 모집중인 스터디는 삭제하는 기능 구현 * refactor: on delete 추가 * chore: flyway v7 * fix: conflict 해소 * test: scenario 위치 수정 * refactor: 이름 수정 및 flyway 쿼리 수정 * refactor: 트랜잭션 통합 * test: sharedcontext 기능 추가 (#387) * test: sharedcontext 기능 추가 * refactor: 잔여 부분 수정 --------- Co-authored-by: Jae Min Yu <[email protected]> * 토큰 만료 확인 기능 (#405) * feat: 토큰 만료 확인 기능 구현 * fix: 테스트 변경 사항에 따른 버그 수정 * feat: 비회원 스터디 상세정보 조회 기능 구현 (#414) * feat: 비회원 스터디 상세정보 조회 기능 구현 - 스터디 상세 조회 API에서 역할을 제공하지 않도록 수정 - 스터디 멤버 역할 조회 API 구현 * feat: 스터디멤버 역할 조회 API 스웨거 적용 * test: 테스트 코드 주석 제거 * refactor: Pattern 객체 상수화 * style: 개행 추가 * test: assertAll 사용하도록 수정 --------- Co-authored-by: kevstevie <[email protected]> Co-authored-by: Jae Min Yu <[email protected]> * fix: flyway 수정 (#415) * Revert "QA 버전 배포 (#413)" (#416) This reverts commit 1bb1174. * 티어 계산 및 경험치 계산 로직 구현 (#397) * feat: 티어 계산 및 경험치 계산 로직 구현 * remove: study에서 최대 보상 계산하는 로직 제거 * refactor: 멤버가 성공적으로 진행한 라운드를 계산하는 기능을 스터디에게 부여 * fix: 없어진 괄호 복구 * test: 깨지는 테스트 수정 --------- Co-authored-by: kevstevie <[email protected]> * 피드 기능을 구현한다. (#404) * test: 피드 조회 기능 테스트 추가 * feat: 피드 조회 기능 추가 * feat: 일반 피드 작성 기능 추가 * feat: 피드 작성 / 조회 기능 추가 * fix: 배열로 시간 직렬화되는 버그 해결 * fix: 인증 및 피드 기능 분리 * Update backend/src/test/java/com/yigongil/backend/acceptance/steps/FeedSteps.java Co-authored-by: Jae Min Yu <[email protected]> * refactor: study service에서 피드를 찾도록 수정 * refactor: pagination에서 cursor를 이용하도록 변경 * fix: flyway추가 --------- Co-authored-by: Jae Min Yu <[email protected]> * refactor: 티어 -> 경험치 flyway 쿼리 추가 (#428) * refactor: 홈뷰 수정 (#430) * refactor: 홈뷰 수정 * refactor: 매직 넘버 수정 * fix: flyway 언더바 수정 * fix: flyway 타입 수정 * refactor: 선택 투두 삭제 (#432) * refactor: 선택 투두 삭제 * refactor: 투두 내용만 수정하게 변경 및 테스트 수정 * refactor: 피드 조회시에 스터디 이름도 반환 (#453) * refactor: 피드 조회시에 스터디 이름도 반환 * Revert "refactor: 피드 조회시에 스터디 이름도 반환" This reverts commit 34b4907. * refactor: conflict 해소 * refactor: aop로 exception handler에서 slack 의존성 제거 (#442) * refactor: aop로 exception handler에서 slack 의존성 제거 * refactor: slack 중복알림 해소 * feat: StudyV1, StudyV2 분리 * feat: Study 엔티티 수정 * fix: 스터디 엔티티 필드 추가 * refactor: PeriodUnit 제거 * 엔티티를 수정하며 발생한 컴파일 에러 해결 (#470) * fix: totalRoundCount, periodOfRound 삭제에 따른 컴파일 에러 해결 * fix: 엔티티 수정에 따른 컴파일 에러 임시 해결 * refactor: swagger 설정 변경 (#452) * refactor: swagger 설정 변경 * refactor: swagger url 설정 파일로 관리 * feat: 동적 쿼리를 통해 스터디 목록 필터링/정렬 (#475) * feat: 동적 쿼리를 통해 스터디 목록 필터링/정렬 * refactor: config v1 prefix 제거 * refactor: 패키지 수정 * refactor: v1 prefix 제거 * refactor: status null 처리 * refactor: response 이름 수정 * refactor: param을 enum으로 컨버팅 * refactor: enum 순서 수정 * refactor: 메서드 추출 * fix: 스터디 시작 할 때 요일을 선택하도록 수정 (#473) * feat: 스터디 개설 기능 구현 #468 * refactor: Study.startAt 제거 및 minimumWeeks와 meetingDaysCountPerWeek 추가 * fix: 스터디 목록 조회 API url 수정 * fix: 실패하는 스터디 일부 복구 및 주석 작성 * test: 실패하는 테스트 복구 * 스터디 상세조회 구현 & 멤버 인증 정보 조회 api 수정 (#484) * refactor: 상세 조회 수정 * feat: 멤버 인증 정보 조회 응답 수정 * feat: 스터디 종료 기능 추가 (#483) * fix: 과거 청산 및 새로운 시작 drop and restart (#490) * feat: 지원한 스터디 목록 조회 기능 (#485) * feat: 지원한 스터디 목록 조회 기능 * refactor: querydsl로 마이그래이션 * chore: 개행 * chore: 정렬 수정 * fix: conflict 해결 중 오류 수정 (#452) * feat: 주차별 회차 조회 기능 추가 (#494) * feat: 주차별 회차 조회 기능 추가 * refactor: 투두 -> 머스트두 변경, 생성 / 수정 통합 * Update backend/src/test/java/com/yigongil/backend/domain/round/RoundTest.java * Update backend/src/test/java/com/yigongil/backend/domain/round/RoundTest.java * fix: 마스터 탈퇴 시 정상적으로 스터디 삭제되게 수정 (#492) * fix: flyway v12 이름 수정 * fix: flyway v12 ddl 수정 * fix: flyway v12 ddl 수정 * fix: swagger 설정 변경 * fix: swagger 설정 변경 * fix: swagger redirect * Revert "fix: swagger redirect" This reverts commit a1cfaa8. * fix: swagger disable efault url * fix: swagger 설정 변경 * fix: flyway v12 이름 수정 * fix: flyway v12 ddl 수정 * fix: flyway v12 ddl 수정 * fix: swagger 설정 변경 * fix: swagger 설정 변경 * fix: swagger redirect * Revert "fix: swagger redirect" This reverts commit a1cfaa8. * fix: swagger disable efault url * fix: swagger 설정 변경 * feat: graceful shutdown (#510) Co-authored-by: Jae Min Yu <[email protected]> * chore: datadog 의존성 추가 * feat: s3 이미지 업로드 기능 추가 (#464) * feat: s3 이미지 업로드 기능 추가 * fix: 이미지 업로드 관련 빈이 로컬에서 로드되지 않도록 변경 * fix: url 수정 * fix: 스터디 멤버 삭제 로직을 study 안으로 이관 (#528) * fix: s3 directory 설정 수정 (#521) * feat: s3 이미지 업로드 기능 추가 * fix: 이미지 업로드 관련 빈이 로컬에서 로드되지 않도록 변경 * fix: url 수정 * fix: s3 directory 수정 * feat: 스터디 탈퇴 기능 (#526) Co-authored-by: Bojun Kim <[email protected]> * feat: 대기중인 스터디 조회 기능 구현 (#531) Co-authored-by: Bojun Kim <[email protected]> * refactor: 인증 조회 방식 수정 (#520) * refactor: 인증 조회 방식 수정 * refactor: optional로 수정 --------- Co-authored-by: Bojun Kim <[email protected]> * fix: collection에서 원소를 삭제하지 않도록 수정 (#537) * fix: collection에서 원소를 삭제하지 않도록 수정 (#537) * hotfix: 이미지 컨트롤러 url 매핑 수정 (#539) * fix: collection에서 원소를 삭제하지 않도록 수정 * fix: ImageResourceController url매핑 수정 * hotfix 배포 (#540) * fix: collection에서 원소를 삭제하지 않도록 수정 (#537) * hotfix: 이미지 컨트롤러 url 매핑 수정 (#539) * fix: collection에서 원소를 삭제하지 않도록 수정 * fix: ImageResourceController url매핑 수정 --------- Co-authored-by: Bojun Kim <[email protected]> * fix : 홈에서 해당 스터디의 누적잔디 조회 가능 (#544) * refactor: 누적 잔디 조회 기능 수정 * chore: todo 제거 * fix: 클라이언트 반환 이미지 경로 수정 (#546) Co-authored-by: Bojun Kim <[email protected]> * qa 버그 수정 버전 배포 (#548) * fix: collection에서 원소를 삭제하지 않도록 수정 (#537) * hotfix: 이미지 컨트롤러 url 매핑 수정 (#539) * fix: collection에서 원소를 삭제하지 않도록 수정 * fix: ImageResourceController url매핑 수정 * fix : 홈에서 해당 스터디의 누적잔디 조회 가능 (#544) * refactor: 누적 잔디 조회 기능 수정 * chore: todo 제거 * fix: 클라이언트 반환 이미지 경로 수정 (#546) Co-authored-by: Bojun Kim <[email protected]> --------- Co-authored-by: kevstevie <[email protected]> Co-authored-by: Jae Min Yu <[email protected]> * [BE] Round 업데이트 로직 수정 (#591) * fix: Round 업데이트 로직 수정 * refactor: 매개변수명 수정 * [BE] refactor: 인증, 피드 최대 글자수 증가 (#599) * refactor: 인증, 피드 최대 글자수 증가 * refactor: 피드 내용 nullable false 적용 --------- Co-authored-by: kevstevie <[email protected]> Co-authored-by: bjk1649 <[email protected]> Co-authored-by: kevstevie <[email protected]> Co-authored-by: kim daehee <[email protected]> Co-authored-by: kim dae hee <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
😋 작업한 내용
🙏 PR Point
👍 관련 이슈
기획의 어떤 부분을 구현 / 수정했는가
다음 스프린트 예정인 s3업로드 기능 미리 구현