-
Notifications
You must be signed in to change notification settings - Fork 0
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
[TNT-115] 연결코드 화면 UI 구현 및 연결 #34
Conversation
- 버튼 text '다음'으로 수정
- 연결 화면에서 홈 화면으로 이동 시 백스택 모두 제거 - 스택 제거 여부는 clearBackStack 플래그로 제어 가능
- TrainerConnectUiState의 page를 TrainerConnectComplete로 변경해 완료 화면 확인
- 글씨가 잘려보이지 않게 width 고정
- 화면 이름들 Page로 변경
- selectedStartDate로 이름 변경 - Calender 안 쓰도록 수정
- pt목적 [] 없이 보여주기 - 주의사항이 없으면 띄우지 않기 - 화면 가운데 정렬
- CodeTextField 파일 분리 - 입력란 상태와 코드 유효성 enum으로 관리 (InputState) - viewModel에서 초대코드 관리하도록 수정
- core:ui에 domain 주입 - UserType별 기본 이미지 제공
- 비율 조정 - 만약 00.0 형태의 실수면 정수로 보여주기 (.0 제거)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍👍👍👍👍
@@ -26,24 +25,27 @@ internal class TrainerConnectViewModel @Inject constructor() : | |||
TrainerConnectUiEvent.OnNextClick -> navigateToNext() | |||
TrainerConnectUiEvent.OnBackClick -> navigateToBack() | |||
TrainerConnectUiEvent.OnSkipClick -> navigateToHome() | |||
else -> {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
else
분기 안해도 될듯염?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
안그래두 뒤에 PR에서 지웠습니다!
if (!trainee.caution.isNullOrEmpty()) { | ||
TextWithBackground( | ||
label = stringResource(R.string.caution), | ||
text = trainee.caution!!, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
non-null assertion
사용하지 않기 !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넵😿 수정해올게요
import co.kr.tnt.trainee.connect.component.InputState.UNFOCUSED | ||
import co.kr.tnt.trainee.connect.component.InputState.VALID | ||
|
||
enum class InputState { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
private
해도 될 것 같아용
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TraineeConnectViewModel
랑 TraineeConnectContract
에서도 사용하고 있어서 public으로 둬야할 것 같습니다
model
로 옮겨둘까요?!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넵넵!
📝 작업 내용
[TNT-115] 파트너 연결 기능 구현 #18
연결 코드 관련 화면들을 구현했습니다 (
trainer
,trainee
로 분리해 구현)트레이너의
트레이니 연결 완료 화면
은 FCM으로 뜨는 화면이기 때문에 이전 화면으로 아무것도 연결되지 않은 상태입니다API는 나오는대로 연결할 예정입니다. 트레이너와 트레이니 프로필 데이터도 API 연결하면서 설정할 예정입니다
📸 실행 화면
트레이너 코드 생성 화면
trainer_code.mp4
트레이너의 트레이니 연결 완료 화면
trainee_profile.mp4
트레이니 연결 화면
trainee.mp4
🙆🏻 리뷰 요청 사항
진짜진짜진짜진짜 죄송합니다.
👀 레퍼런스
https://developer.android.com/develop/ui/views/touch-and-input/copy-paste?hl=ko#duplicate-notifications