Skip to content

Conversation

@soyun-git121
Copy link
Collaborator

@soyun-git121 soyun-git121 commented Aug 5, 2025

🔍 관련된 이슈

📝 작업 내용

  • 프로필수정 페이지 api연결
  • 간격 수정

📸 스크린샷

image image

🚨 이슈

이유는 모르겠으나.. s3이미지 업로드 관련 올려주신 pr이 전체pull이나 pr지정 pull해도 이미 업데이트됐는데
vscode에 나타나지않아서 파일 그대로 복붙한거라 무시해주셔도 됩니당...

📣 리뷰 요구사항

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성

✅ 체크리스트

  • 코드가 정상적으로 컴파일되나요?
  • merge할 브랜치의 위치를 확인했나요?
  • Label을 지정했나요?
  • 리뷰어를 지정했나요?

@soyun-git121 soyun-git121 self-assigned this Aug 5, 2025
@soyun-git121 soyun-git121 added the 🔗API api 관련 작업 label Aug 5, 2025
@soyun-git121 soyun-git121 linked an issue Aug 5, 2025 that may be closed by this pull request
1 task
@soyun-git121 soyun-git121 added the ✨Feature 새로운 기능 추가 label Aug 5, 2025
Comment on lines 7 to 8
const res = await api.put('/api/v1/profile', data);
return res.data.data;
Copy link
Collaborator

Choose a reason for hiding this comment

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

git pull origin develop 하시고 이 부분 '/profile'로 바꿔주셔야 할 것 같습니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

넵!

Copy link
Member

@dalzzy dalzzy left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~😄 시간이 부족해서 빠르게 구현하고 싶은 마음은 충분히 이해댑니다..!!
다만 스웨거랑 피그마를 조금 더 꼼꼼히 보시고, 요구사항에 맞춰 작성해주셔야 할 것 같아요 ㅠㅠ
파이팅입니당

export const updateUserProfile = async (
data: UpdateUserProfileRequest
): Promise<UserProfileResponse> => {
const res = await api.put('/api/v1/profile', data);
Copy link
Member

Choose a reason for hiding this comment

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

이 부분도 .. /api/v1 지워주셔야 할 것 같습니당
그리고 프로필 수정 api 메소드는 put이 아니라 patch입니다 !!!!!! .. ㅜㅜ
image


export const getUserProfile = async (): Promise<UserProfile> => {
try {
const response = await api.get<UserProfile>('/profile');
Copy link
Member

Choose a reason for hiding this comment

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

ApiResponse 타입을 활용하면 좋을 것 같습니당

Comment on lines 53 to 56
showToast('프로필이 저장되었습니다.', 3000);
navigate('/my');
} catch (error) {
showToast('저장에 실패했습니다.', 3000);
Copy link
Member

Choose a reason for hiding this comment

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

image

피그마랑 토스트 메세지가 다른 것 같습니다 수정부탁드릴게요

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

넵 맞춰서 다른부분들도 변경하도록하겠습니다!

@soyun-git121 soyun-git121 changed the title [feat] 프로필 api 연결 [feat] 프로필 수정 api 연결 Aug 6, 2025
Copy link
Member

@dalzzy dalzzy left a comment

Choose a reason for hiding this comment

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

코드 병합이 잘못된 것 같은데,, 한번 다시 확인 부탁드립니다..!

@soyun-git121 soyun-git121 merged commit c9a03e7 into develop Aug 6, 2025
1 check passed
@dalzzy dalzzy deleted the feat/#65/프로필수정-api-연결 branch August 8, 2025 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔗API api 관련 작업 ✨Feature 새로운 기능 추가

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feat] #65 프로필수정 api연결

4 participants