[MOD/#1577] 콕찌르기 천생연분 세션 실명 프로필로 변경#1581
Merged
Merged
Conversation
Modified `isSoulMate` logic to remove the `isAnonymous` check.
- `MessageListBottomSheetFragment`에 익명 체크박스 가시성을 제어하는 `isAnonymousCheckboxVisible` 필드 및 Builder 메서드 추가 - 누적 콕 횟수(`pokeNum`)가 10회 미만인 경우에만 익명 체크박스가 보이도록 `isAnonymousCheckboxVisible` 유틸리티 함수 구현 - `FriendListSummaryActivity`, `FriendListDetailBottomSheetFragment`, `PokeScreen` 등 주요 화면에서 유저의 `pokeNum`에 따라 익명 옵션을 동적으로 노출하도록 수정 - 익명 체크박스가 숨겨진 경우 익명 여부(`isAnonymous`)를 `false`로 전달하도록 처리
- `isAnonymousVisible` 유틸 함수를 추가하여 익명 여부와 관계명(SOULMATE 제외)을 동시에 확인하도록 로직 개선. - `PokeUserViewHolder`, `PokeNotificationAdapter`, `PokeUserListAdapter` 및 `PokeScreen` 내 프로필 이미지 로드, 이름 표시, 클릭 리스너 로직에 해당 조건 적용. - 천생연분(SOULMATE) 관계일 경우 익명 상태여도 실명이 노출되도록 수정.
seungjunGong
requested review from
1971123-seongmin,
Hyobeen-Park,
sonms and
vvan2
May 12, 2026 15:48
sonms
approved these changes
May 13, 2026
sonms
left a comment
Member
There was a problem hiding this comment.
굿굿 고생하셨습니다!!! 이제 천생연분 시에도 잘 보이겠네요~
- `MessageListBottomSheetFragment`에서 익명 체크박스의 가시성을 설정하던 `isAnonymousCheckboxVisible` 관련 코드 및 Builder 메서드 제거 - `FriendListSummaryActivity`, `FriendListSummaryScreen`, `FriendListDetailBottomSheetFragment`, `PokeScreen` 등에서 호출하던 관련 유틸 로직 제거 - `PokeUtil.kt` 내 사용되지 않는 `isAnonymousCheckboxVisible` 함수 및 `soulMateMinPokeCount` 상수 삭제
- 익명 정보 노출 여부를 판단할 때 관계명(`relationName`) 체크 없이 `isAnonymous` 필드만 직접 확인하도록 수정하였습니다. - 사용되지 않는 `isAnonymousVisible` 유틸리티 함수를 제거하였습니다. - `PokeUserViewHolder`, `PokeNotificationAdapter`, `PokeUserListAdapter`, `PokeScreen` 등 관련 클래스에서 변경된 로직을 적용하였습니다.
- `MessageListBottomSheetFragment`에 익명 체크박스 잠금 상태(`isAnonymousCheckboxLocked`) 필드 및 설정 로직 추가. - `pokeNum`이 10회 이상일 경우 익명 체크박스를 비활성화하고 실명 전송만 가능하도록 변경. - 익명 기능이 잠긴 상태에서 체크박스 클릭 시 안내 토스트 메시지 출력. - `PokeUtil`에 `isAnonymousCheckboxLocked` 판단 로직 및 관련 상수 추가. - `PokeScreen`, `FriendListSummaryActivity`, `NotificationScreen` 등 주요 화면에서 해당 잠금 로직 적용.
- `isAnonymousVisible` 유틸 함수를 추가하여 천생연분 단계 진입 시(찌른 횟수 11회) 익명 해제 로직 반영 - `PokeUserViewHolder` 및 `PokeUserListAdapter`에서 단순 `isAnonymous` 필드 체크 대신 `isAnonymousVisible`을 사용하도록 수정
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
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.
Related issue 🛠
Work Description ✏️
-> 11번째 콕에서만 익명 여부를 무시하고 실명으로 표시되도록 수정
Screenshot 📸
<<최종 버전>>
Screen_recording_20260516_140433.mp4
Uncompleted Tasks 😅
N/A
To Reviewers 📢
활성화된 콕찌르기를 다써서 영상에는 10번째 콕인경우에대한 부분과 애니메이션 처리에 대한 시연은 없는데,,
한번씩은 다 해보았습니다..! 혹시 이상있는경우 알려주시면 감사하겠습니다 ~
++ 추가로 논의된 부분도 작업했어요.
회의때 이야기한것처럼 기존에 프로필이 다보이는 것 보다는 앞으로 콕찌르기를 할때
실명으로 찌르게 하면 마이그레이션이 더 자연스러울 거라 생각해서 프로필이 다보이게 하지 않고 API 를 실명으로 찌르게 변경했습니다.