Skip to content

Conversation

@0-hu
Copy link
Contributor

@0-hu 0-hu commented May 28, 2025

📌 관련 이슈번호

✏️ PR 유형

어떤 변경 사항이 있나요?

  • 새 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 리팩토링

✅ Check List

  • 🔀 PR 제목의 형식을 잘 작성했나요?
  • 🏗️ yarn build는 성공했나요?
  • 🧹 불필요한 코드는 제거했나요? (TODO, 주석, clg... etc.)
  • ✅ 컨벤션을 지켰나요?
  • 💭 이슈는 등록했나요?
  • 🏷️ 라벨은 등록했나요?
  • 🙇‍♂️ 리뷰어를 지정했나요?

✨ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

  1. 작업 내용

마이페이지 API 연결

  • 지금 마이페이지 ui에서 보여주는 내용이 별로 없어서 이름/글로우만 붙였습니다.

마이페이지 온도(글로우) 변경

  • 일단 그냥 맘대로 바꾸긴 했는데, 시연 영상에 포함하기 애매하거나 하면 요거 제외할게요.

📢 To Reviewers

📸 스크린샷

온도 변동
스크린샷 2025-05-28 213712

온도 레벨
image

image

🔗 참고 자료

Copy link
Contributor

@sinji2102 sinji2102 left a comment

Choose a reason for hiding this comment

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

프로필 이미지 수정 API 오류 나는 것 같아요~!
image

그리고 서버 전송하기 전에 사용자가 넣은 사진 바로 화면에서 보여주는 기능도 있어야 할 것 같습니다!

그리고 글로우 온도 디자인/서버 측이랑도 논의해봐야 할 것 같아요! 지금 디자인도 통일되지 않고 이러면 서버에서 오는 값도 달라져야 해서...

Comment on lines +98 to +106
<input
type="text"
value={currentNickname}
onChange={handleNicknameInputChange}
onBlur={handleNicknameBlur}
className="body-lg text-layout-grey7 h-full w-full bg-transparent outline-none disabled:opacity-50"
placeholder="닉네임을 입력하세요"
disabled={isNicknameUpdating}
/>
Copy link
Contributor

Choose a reason for hiding this comment

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

p1) 닉네임 변경하면 로컬스토리지에도 반영하도록 해 주세요!

그리고 이건 디자인 이슈긴 한데 닉네임 변경 버튼이 따로 있어야 할 것 같아요!

@0-hu
Copy link
Contributor Author

0-hu commented May 29, 2025

요거 에러나는건 확인좀 해보고 승원님이랑 온도 관련해서는 모달까지 다 정리한다음에 수정하겠습니다

영상은 요 pr 빼고 지금까지 싱크 맞춰놓은걸로 찍을게요

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.

[ Feat ] 마이페이지 API 연동

3 participants