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.

Sign up for GitHub

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

Jump to bottom

feat: 회원 프로필 이미지 기능 구현 #78

Merged
merged 20 commits into from
Oct 7, 2024
Merged

feat: 회원 프로필 이미지 기능 구현 #78

merged 20 commits into from
Oct 7, 2024

Conversation

junest66
Copy link
Collaborator

@junest66 junest66 commented Oct 5, 2024 •
edited
Loading

#️⃣연관된 이슈

CHZZ-128

📝작업 내용

  • 회원의 프로필 이미지 기능을 추가하였습니다.
  • 프로필 조회 API, 프로필 수정 API, 경매 상세정보 조회 API, 사전 경매 상세정보 조회 API 를 수정하였습니다.(postman 업데이트 완료)
  • 프로필 이미지 필드로인한 flyway script 추가하였습니다.
  • 프로필 조회시 토큰 기반 조회시에만 providerType을 추가하였습니다.

✅테스트 결과

스크린샷 2024-10-06 오전 2 20 57

스크린샷 2024-10-06 오전 2 20 17

🙏리뷰 요구사항(선택)

  • 제가 노션에 API 업데이트 내역 추가해뒀습니다. 어자피 변경사항 프론트분들한테 말해야하니깐 차라리 적어서 확인하는게 나을 것 같아 추가해뒀습니다.(postman 업데이트+ api 업데이트 내역)

@junest66
feat: 회원 테이블에 프로필 이미지 url 컬럼 추가
7c21e9d 
CHZZ-128
@junest66
feat: 회원 테이블에 프로필 이미지 url 컬럼 추가로 인한 flyway 스크립트 추가
2cd2c60 
CHZZ-128
@junest66
feat: 프로필 수정 DTO에 기본이미지를 사용할지 여부 필드 추가
06299de 
CHZZ-128
@junest66
refactor: imageService uploadImage 함수 리턴값 수정
6529730 
- 부분 경로에서 전체 경로 반환값으로 수정

CHZZ-128
@junest66
refactor: 회원수정에 프로필 이미지 추가로 인한 서비스 함수 수정
b58e743 
CHZZ-128
@junest66
test: 회원수정에 프로필 이미지 추가로 인한 서비스 함수 테스트 코드 수정
0add174 
CHZZ-128
@junest66
refactor: 회원수정 컨트롤러 요청 값 수정
513b9cf 
- MultipartFile 객체 추가

CHZZ-128
@junest66
style: 가독성을 위한 함수 순서 변경
0bce3e2 
CHZZ-128
@junest66
delete: 쓰지 않는 파일 제거
22c33c0 
CHZZ-128
@junest66
refactor: 경매 상세정보 응답에 판매자의 프로필 이미지 필드 추가
c90f6f4 
CHZZ-128
@junest66
refactor: 경매 상세정보 응답에 판매자의 프로필 이미지 필드 추가로 인한 쿼리 수정
6773217 
CHZZ-128
@junest66
test: 경매 상세정보 응답에 판매자의 프로필 이미지 필드 추가로 인한 테스트 코드 수정
6ab6987 
CHZZ-128
@junest66
refactor: 사전 경매 제품 상세정보 응답에 판매자의 프로필 이미지 필드 추가
6a573ab 
CHZZ-128
@junest66
refactor: 사전 경매 제품 상세정보 응답에 판매자의 프로필 이미지 필드 추가로 인한 쿼리 수정
1b6c770 
CHZZ-128
@junest66
refactor: 프로필 조회 응답 DTO에 프로필 이미지 url 추가
831806b 
CHZZ-128
@junest66
style: 가독성을 위한 함수 순서 수정
8ec73b3 
CHZZ-128
@junest66
refactor: 프로필 조회에 providerType 추가
45c4ef3 
- 토큰 기반 응답 시 providerType 포함
- 닉네임 기반 응답시 개인정보 보안으로 미 포함

CHZZ-128
@junest66
refactor: 프로필 조회에 providerType 추가로 인한 서비스 함수 수정
c373f8b 
CHZZ-128
@junest66 junest66 added the ✨feature 구현, 개선 사항 관련 부분 label Oct 5, 2024
@junest66 junest66 self-assigned this Oct 5, 2024
viaunixue
viaunixue approved these changes Oct 7, 2024
View reviewed changes
Copy link
Member

@viaunixue viaunixue left a comment

Choose a reason for hiding this comment

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

프로필 이미지 변경 쪽 로직 모두 확인했습니다!

@junest66 junest66 merged commit dd7ffda into JECT-Study:develop Oct 7, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@YeaChan05 YeaChan05 YeaChan05 approved these changes

@viaunixue viaunixue viaunixue approved these changes

Assignees

@junest66 junest66

Labels
✨feature 구현, 개선 사항 관련 부분
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@junest66 @viaunixue @YeaChan05