[TNT-190] 트레이니 연결 요청 팝업 기능 구현 #110
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- '3일간 보지 않기' 기능 구현 필요
- 현재 완료된 회차에 0 입력 가능하도록 조건 수정
hoyahozz
approved these changes
Feb 14, 2025
| class ConnectLocalDataSource @Inject constructor( | ||
| @Named(CONNECT_STORAGE_NAME) private val connectPreferences: DataStore<Preferences>, | ||
| ) { | ||
| val homeDialogHiddenDate: Flow<String?> = connectPreferences.data.map { preferences -> |
Member
There was a problem hiding this comment.
변수명이 UI와 의존성을 지니지 않도록
ExplicitDeniedConnectDate 와 같은 방식으로 네이밍하는게 좋을 것 같아요!
Contributor
Author
There was a problem hiding this comment.
앗 넵 수정했습니다!
|
|
||
| val lastHiddenDate = connectRepository.getHomeDialogHiddenDate().firstOrNull() | ||
| val isHidden = lastHiddenDate != null && | ||
| Duration.between(lastHiddenDate, currentDateTime).toHours() < 72 |
| @Composable | ||
| private fun ProfileSection( | ||
| profile: User, | ||
| context: Context, |
Member
There was a problem hiding this comment.
추후에 painter 받아서 사용하는 쪽에서 placeholder 설정하는 방식으로 만들면 좋을 것 같아요
그럼 context 도 안받아도 될듯요!
# Conflicts: # feature/trainee/signup/src/main/java/co/kr/tnt/trainee/signup/TraineeSignUpViewModel.kt # feature/trainer/signup/src/main/java/co/kr/tnt/trainer/signup/TrainerSignUpViewModel.kt
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📝 작업 내용
Closes [TNT-190] 트레이너 연결 요청 팝업 #53
홈 화면 진입 시 연결 요청 팝업이 자동으로 표시되도록 구현했습니다.
연결되지 않은 상태이고3일 동안 보지 않기로 설정한 시간이 지났다면다이얼로그를 표시하도록 처리했습니다.
3일 동안 보지 않기 기능을 구현했습니다.
3일 동안 보지 않기를 선택하면 다이얼로그가 닫히는 순간의 LocalDateTime을ConnectLocalDataSource에 저장하도록 구현했습니다.회원가입 및 프로필 사진과 관련된 수정 사항 포함되어 있습니다
📸 실행 화면
트레이너 연결 이후 팝업 노출
default.mp4
3일동안 보지 않기 동작
3.mp4
🙆🏻 리뷰 요청 사항
없습니다.. 영상에 보이는 연결 완료 화면 이미지 오류는 해결했습니다!
👀 레퍼런스