Skip to content

[refactor] - 마이페이지 디자인 완성#45

Open
JI00000 wants to merge 9 commits intodevelopfrom
jiyoung_refactor/#38
Open

[refactor] - 마이페이지 디자인 완성#45
JI00000 wants to merge 9 commits intodevelopfrom
jiyoung_refactor/#38

Conversation

@JI00000
Copy link
Copy Markdown
Collaborator

@JI00000 JI00000 commented Aug 19, 2022

[수정사항]

  • 마이페이지 전체 디자인 변경
  • 마이페이지 전체 레이아웃 아이디 변경

[확인사항]

  • 마이페이지 전체 레이아웃 및 아이디 변경사항 확인 요청

  • activity myaccount

  • fragment mypage

  • activity changenickname

  • activity changepassword

  • fragment scrapalcohol

  • fragment scrappost

  • activity writtenpost

  • activity writtencomment

  • activity setting

  • activity addalcohol

  • dialog addalcohol

[작업번호]
#38

[참고 이미지]

@JI00000 JI00000 added the refactor 코드 리팩토링 label Aug 19, 2022
@JI00000 JI00000 self-assigned this Aug 19, 2022
@gksgpd97
Copy link
Copy Markdown
Collaborator

수정사항 확인하려면 어느 커밋들 보면 될까요?
커밋 메세지 작성하실 때 마지막에 #00 으로 이슈번호 붙여주시면 더 편하게 리뷰할 수 있을 것 같습니다!

@JI00000
Copy link
Copy Markdown
Collaborator Author

JI00000 commented Aug 21, 2022

넵넵!! 다 디자인 관련이라 따로 번호는 붙이지 않았었는데 앞으로 커밋 후에는 번호로 구분지어 놓겠습니다!! [3768c9a], [0a7f1c2] 확인해주시면 됩니다~! 아 그리고 질문이 있는데 혜원님께서는 pull하신 후에 commit 하고 다시 작업을 이어나가시는 편이실까요?? 제가 변경한 부분이랑 pull하면서 변경된 부분이 변동사항으로 함께 올라가다보니 코드 보실때 헷갈리실 수도 있을 거 같아서요!

@songfox00
Copy link
Copy Markdown
Collaborator

코드리뷰 완료했습니다~ 라인에 리뷰 달 수 있는 걸... 지금 알았네요...ㅋㅋㅋ

@gksgpd97
Copy link
Copy Markdown
Collaborator

@JI00000 리뷰 늦어져서 죄송합니다. 라인마다 피드백 달았습니다. develop과 pull하는 걸 말씀하신거죠? 네 저는 커밋하고 이어갑니다! 코드 볼 때 헷갈리는 부분은.. 저도 생각해보지 못했는데 불편한 게 생기면 저도 말씀드릴테니 여러분도 말씀해주세요.
고생하셨습니다~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor 코드 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants