Feat/logout #40
Feat/logout #40
Conversation
| } else { | ||
| Text(stringResource(id = R.string.nickname_qualification)) |
There was a problem hiding this comment.
HE 피드백 반영하여 회원가입 시 닉네임 조건 미리 고지하는 걸로 수정해둔 부분이라 이 라인들 삭제하지 않고 그대로 둬야할 것 같아요!
PR #37 참고해주세욥
| supportingText = { | ||
| if (isPasswordError) { | ||
| Text(stringResource(id = uiState.error!!.messageId)) | ||
| } | ||
| }, |
There was a problem hiding this comment.
이 부분도 위 코멘트와 마찬가지로 회원가입 시 비밀번호 조건 미리 고지하는 걸로 수정해 둔 부분이라
디폴트로 supportingText 제공해줘야할 것 같아요!
PR #37 참고해주세욥
There was a problem hiding this comment.
progress indicator 들어간 거 완전 좋네요👍👍
| messageId = R.string.nickname_qualification | ||
| ) | ||
| ) | ||
| } |
There was a problem hiding this comment.
isValidNickname()이 isNotBlank() && length <= 15 인지 체크하고 있고 R.string.nickname_length_error가 "닉네임을 입력해주세요"라서, 현재 닉네임이 15자를 초과하면 "닉네임을 입력해주세요"라는 에러 메시지가 유저에게 보여지게 됩니다
--> R.string.nickname_length_error을 "닉네임은 1자 이상 15자 이하여야 합니다"로 변경해야할 것 같아요!
(validation을 blank 유무, 15자 초과 유무로 나누는 것 보다 string만 바꾸는 게 나을 것 같네욥!!)
|
민영님 석찬님 너무너무 고생많으셨습니다!!!!!🔥🔥 PR #37 머지 이후에 작업을 시작해주셨는데 그래서 이후 수정사항 반영해주시기 전에 꼭 pull 한번 해주세욥! 이 외에는, 현재 아래와 같은 현상들이 보이는데
요 부분이 해결돼야 메인에 머지할 수 있을 것 같습니다..!! |
PR Title: Implement logout and withdraw feature
Related Issue(s):
PR Description:
Changes Included:
Notes for Reviewer:
Reviewer Checklist:
Additional Comments: