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

feat: UserInformationCard에 현재 로그인 중인 사용자 정보 넣기 #232

Merged
merged 2 commits into from
Jun 25, 2024

Conversation

nijuy
Copy link
Collaborator

@nijuy nijuy commented Jun 23, 2024

1️⃣ 어떤 작업을 했나요? (Summary)

기존 코드에 영향을 미치지 않는 변경사항

김뿌슝씨를 쫓아냈습니다

image

스웨거로 imagePK 확인하다가 갈매기 사진으로 바뀌었는데 일단 사용자는 프로필 변경 못 하게 해뒀으니 신경쓰지 마세요..!

2️⃣ 알아두시면 좋아요!

  • 스웨거에 프로필 수정 api가 존재하는데 여기서 작업 안 한 이유

    프로필 이미지 / 닉네임 수정 API가 별도로 존재하는 게 아니라 하나만 있는데요,
    제가 알기론 사용자가 이미지 파일을 직접 업로드 하는 게 아니라 번호만 고르는 식인 걸로 알거든요..?
    (뿌슝이 이미지 N개 중 사용자가 선택 -> 그 번호를 imagePK에 넣음 -> 서버에서 imagePK에 맞는 뿌슝이 이미지를 프로필로 설정)

    근데 지금 뿌슝이 이미지가 준비된 상태도 아니고, 어떻게 선택하게 할지에 대한 뷰도 없어서
    일단 프로필 편집 버튼을 전부 주석처리 해뒀습니다 (혹시 제가 잘못 알고있는 거면 알려주세요)

    홈 TF 채널에 프로필 이미지 관련 스레드가 있었는데 기간 지나서 안 보여요 흑흑 한번 더 물어보겠습니당...
    어쨌든 김뿌슝씨 그대로 두긴 좀 그래서 일단 올립니다

    image

3️⃣ 추후 작업

4️⃣ 체크리스트 (Checklist)

  • main 브랜치의 최신 코드를 pull 받았나요?

- 프로필 수정 기능에 접근하지 못하게 주석 처리 해둠
@nijuy nijuy self-assigned this Jun 23, 2024
@nijuy nijuy linked an issue Jun 23, 2024 that may be closed by this pull request
1 task
@nijuy nijuy added feat New feature or request home labels Jun 23, 2024
Copy link
Member

@seocylucky seocylucky left a comment

Choose a reason for hiding this comment

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

김뿌슝씨 드디어 수정이 됐군요
확인했습니다! 정보 적용 잘되네용

궁금한게 엄청난 뒷북일 수 있지만 imageUrl에 large mid small의 차이가 뭔가요?

Copy link
Member

@seocylucky seocylucky left a comment

Choose a reason for hiding this comment

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

제가 알기론 사용자가 이미지 파일을 직접 업로드 하는 게 아니라 번호만 고르는 식인 걸로 알거든요..?
(이미지 N개 중 사용자가 선택 -> 그 번호를 imagePK에 넣음 -> 서버에서 imagePK에 맞는 이미지를 프로필로 설정)

이 부분 제대로 이해를 못했는데.. 웹이나 앱에서 사용자 정보를 수정할 때 갤러리 같은 시스템에서 이미지를 고를 텐데 사진 별로 imagePK(사진 고유 키값)가 있는건가요? 번호만 고른다는 방식을 이해를 못했..서요

@nijuy
Copy link
Collaborator Author

nijuy commented Jun 24, 2024

@seocylucky

궁금한게 엄청난 뒷북일 수 있지만 imageUrl에 large mid small의 차이가 뭔가요?

없.. 없습니다
과거 백엔드 스펙에 맞추느라 필드 3개 그대로 냅둔거고, 숀이 나중에 줄여야겠다고 했던 메시지를 슬랙 어딘가에셔 봤는데 지금 90일 지나서 안 보일거에요………..🥹

웹이나 앱에서 사용자 정보를 수정할 때 갤러리 같은 시스템에서 이미지를 고를 텐데

아하 이게 안된단 얘기였어요!
사용자가 원하는 이미지를 자유롭게 업로드하는 게 아니라, 주어진 프로필 사진 n개 중에 하나를 고르는 식인걸로 안다는 말이었습니다

Copy link
Collaborator

@jonique98 jonique98 left a comment

Choose a reason for hiding this comment

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

김뿌슝씨를 쫓아냈습니다

좋은 곳으로 가셨으리라 생각합니다.
고생하셨습니다.

Copy link
Member

@2wndrhs 2wndrhs left a comment

Choose a reason for hiding this comment

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

수고하셨습니다👍

const [activeEditMode, setAcitveEditMode] = useState(false);
const [nickname, setNickname] = useState(Dummy.name);
const [nickname, setNickname] = useState<string>('');
Copy link
Member

@2wndrhs 2wndrhs Jun 25, 2024

Choose a reason for hiding this comment

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

이거 저만 ESLint 오류 발생하나요..?

해당 라인이랑 아래 handleInputChange 부분도 같이 주석처리 해줘야 될 것 같아요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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


수정했습니다!!1

@nijuy nijuy merged commit 66c603a into develop Jun 25, 2024
@nijuy nijuy deleted the feat/#231-mypage-userinfo branch June 25, 2024 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or request home
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] 마이페이지 유저 정보 조회
4 participants