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

M3-168 유저정보 수정 #30

Merged
merged 9 commits into from
Jul 19, 2024
Merged

Conversation

tkdals802
Copy link
Contributor

@tkdals802 tkdals802 commented Jul 18, 2024

작업 내용*

  • 사진을 전송할 수 있도록 유저 정보 수정 서비스 수정
  • 유저 정보 입력, 수정 페이지의 키보드 바깥을 누르면 키보드가 내려가도록
  • 처음에 들어가면 유저의 정보를 띄워두도록 한다(닉네임, 생년, 성별, 프로필사진)
  • 프로필사진이 존재하면 s3에서 가져온다, 없으면 기본 프로필 이미지를 띄운다
  • 파일에서 이미지를 선택하면 해당 이미지가 나타나도록 한다
  • 정규식을 사용해 형식에 맞지 않으면 동적으로 텍스트창 아래에 문구가 나타나도록 + 확인 버튼 누를 시 경고창 생성
  • 닉네임 중복시 경고창 생성

고민한 내용*

  • 사진을 multipartFile로 전송하기
  • s3url로 NetworkImage로 사진 받아오기
  • formdata로 data전송하기
  • get으로 받아온 user정보를 초기에 띄우기
  • 정규식을 사용해 동적으로 경고문 띄워주기

리뷰 요구사항

  • 함수, 변수 이름, 로직

스크린샷

| image |

@tkdals802 tkdals802 requested review from qjvk2880 and koomin1227 and removed request for qjvk2880 July 18, 2024 04:28
Copy link
Member

@koomin1227 koomin1227 left a comment

Choose a reason for hiding this comment

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

지금 보니까 닉네임이 place holder 로 채워져있어서 출생년도 만 바꾸고 싶을때 완료 버튼이 안눌리네요.

중복 처리도 추가해주시면 감사하겠습니다. 400 응답 올때 중복인 것을 표시하면 될 것 같네요

@tkdals802 tkdals802 changed the title Feature/m3 168 user info update M3-168 유저정보 수정 Jul 19, 2024
@tkdals802 tkdals802 force-pushed the feature/M3-168-userInfoUpdate branch from cfa5914 to c6ea7ee Compare July 19, 2024 08:36
@tkdals802 tkdals802 merged commit 46edc52 into develop Jul 19, 2024
1 check passed
@tkdals802 tkdals802 deleted the feature/M3-168-userInfoUpdate branch July 19, 2024 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants