[Fix] 1차 QA 수정사항 반영 (로그인/회원가입 · 홈 · 설정) - #218
Merged
Hidden character warning
The head ref may contain hidden characters: "FLT-26-1\ucc28-qa-\uc218\uc815"
Merged
Conversation
QA TC 1-34 - 한글/영문/숫자 혼합 입력 시 에러가 발생하던 문제 수정 NICKNAME_REGEX에 0-9가 빠져 있어 숫자가 포함된 닉네임이 거부됐습니다. 온보딩과 프로필 수정 화면에 동일한 정규식이 복사돼 있어 함께 수정했습니다. (온보딩만 고치면 가입은 되는데 수정은 거부되는 불일치가 생깁니다) Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
QA TC 3-23, 3-24 - 저장한 콘텐츠가 10개를 넘어도 계속 노출되던 문제 수정 getUserBookmarkedContents()는 프로필/저장한 콘텐츠 화면에서도 사용하므로 Repository가 아닌 HomeViewModel에서 잘랐습니다. totalCount는 프로필 쪽에서 쓰이므로 전체 개수를 유지합니다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
QA TC 12-6, 12-7 - 계정 정보 영역에 이메일이 표시되지 않던 문제 대응 스웨거 MyProfileRes(GET /api/v1/users/me)에 email이 문서화돼 있으나 UserProfileResponseDto에 필드가 없어 값이 와도 받을 수 없는 상태였습니다. DTO/모델/매퍼/UiState까지 email을 연결하고 UI에 노출합니다. 단, 실기기 확인 결과 서버가 실제로는 email을 내려주지 않습니다 (null인 필드를 응답에서 생략하는 구조라 키 자체가 없음). 따라서 이 커밋만으로 TC 12-6/12-7이 닫히지는 않으며, 카카오 이메일 동의항목 수집 여부에 대한 백엔드 확인이 필요합니다. 이메일이 있으면 노출하고 없으면 로고만 표시하는 구조로 정리했습니다. 타 유저 응답(UserProfileRes)에는 email이 없고 DTO를 공유하므로 keywordRecalculatable과 동일하게 nullable + 기본값으로 처리했습니다. 긴 이메일이 "계정" 라벨을 밀어내지 않도록 trailingContent를 RowScope로 변경하고 weight(1f, fill = false)를 적용했습니다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
서버 스키마 GetOttResponse는 { ottId, name, logoUrl } 세 필드뿐이라
기본값 없는 contentUrl이 있으면 MissingFieldException으로 역직렬화가 실패합니다.
현재는 서버가 빈 배열을 주고 있어 드러나지 않지만,
실제 OTT 데이터가 채워지는 순간 홈 바텀시트가 깨집니다.
ignoreUnknownKeys/coerceInputValues로는 막을 수 없어(누락 필드는 대상 아님)
기본값을 지정했습니다.
함께 추가한 유닛 테스트:
- OttListResponseDtoTest: contentUrl 유무 양쪽 역직렬화 (4개)
- NicknameValidationTest: 닉네임 숫자 허용 및 거부 규칙 (12개)
온보딩과 프로필 수정의 판정이 일치하는지도 함께 검증
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
QA TC 3-33, 3-34 - 홈에서 저장 콘텐츠를 눌러도 바텀시트가 뜨지 않던 문제
원인은 서버가 GET /api/v1/contents/ott/{contentId} 에 대해
{"otts":[]} 빈 배열만 반환하는 것이었습니다.
빈 목록이면 HomeScreen 의 isNotEmpty() 가드에 걸려 아무 반응이 없었습니다.
반면 북마크 목록 응답은 같은 콘텐츠에 대해 getOttSimpleList 를 정상적으로
내려주고 있고, 프로필(ProfileScreen)과 저장한 콘텐츠(SavedContentScreen)는
이미 이 값을 사용합니다. 홈만 별도 API 를 호출하고 있어 홈에서만 실패했습니다.
홈도 동일하게 이미 로드된 북마크 목록의 getOttSimpleList 를 사용하도록 변경했습니다.
바텀시트는 OttType 의 로컬 iconRes/ottName 으로 렌더링하므로
서버의 logoUrl(현재 "adsf" 등 더미값)과 contentUrl 은 필요하지 않습니다.
부수 효과:
- 콘텐츠 탭마다 발생하던 네트워크 호출이 사라짐
- OttShortCutListItem 의 OttType.valueOf() 는 runCatching 없이 호출되는데,
로컬 값은 이미 OttType 으로 검증된 값이라 IllegalArgumentException 위험이 없어짐
- 미사용이 된 ContentRepository 의존성 제거
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
🚧 Files skipped from review as they are similar to previous changes (3)
📝 WalkthroughWalkthroughOTT 응답 기본값과 저장 콘텐츠의 OTT 목록 표시 흐름을 변경했습니다. 사용자 프로필 이메일을 설정 화면에 연결했습니다. 온보딩과 프로필 수정 화면에서 숫자를 포함한 닉네임을 허용하고 관련 테스트를 추가했습니다. ChangesOTT 목록 처리
프로필 이메일 연동
닉네임 검증
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant HomeScreen
participant HomeViewModel
participant ContentModel
participant ShowOttListBottomSheet
HomeScreen->>HomeViewModel: showOttList(contentId)
HomeViewModel->>ContentModel: 저장된 콘텐츠에서 contentId 검색
ContentModel-->>HomeViewModel: ottSimpleList 반환
HomeViewModel->>ShowOttListBottomSheet: OTT 목록 표시 이벤트 전달
Possibly related PRs
Suggested labels: Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@app/src/main/java/com/flint/presentation/home/HomeViewModel.kt`:
- Around line 80-82: Update getUserBookmarkedContents() and the
bookmarked-content flow so contents are ordered by save date descending before
take(MAX_SAVED_CONTENT_COUNT), or add the save-date field to the response DTO to
enable that ordering. Ensure the resulting home list always contains the latest
saved 10 items, and keep the immutable-list conversion after limiting.
In `@app/src/main/java/com/flint/presentation/setting/SettingScreen.kt`:
- Around line 117-130: 중첩된 Row가 SettingMenuItem의 가용 폭을 사용하도록 수정하고, 그 안의 email
Text가 남은 공간만 차지하게 하여 TextOverflow.Ellipsis가 동작하도록 하세요. SettingMenuItem의 제한된 폭과
아이콘 영역을 침범하지 않도록 Row의 weight 또는 동등한 폭 제약을 적용하고, Text의 기존 말줄임 설정은 유지하세요.
In
`@app/src/test/java/com/flint/presentation/onboarding/NicknameValidationTest.kt`:
- Around line 49-51: Update the test input in `한글 영문 숫자를 모두 섞은 닉네임을 허용한다` to a
2–8-character nickname that mixes Korean, English, and digits, so it satisfies
both UI states’ `MAX_LENGTH` constraint while preserving the intended validation
coverage.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 6d2a7556-7db1-4401-afa4-33e160bb4db8
📒 Files selected for processing (13)
app/src/main/java/com/flint/data/dto/ott/response/OttListResponseDto.ktapp/src/main/java/com/flint/data/dto/user/response/UserProfileResponseDto.ktapp/src/main/java/com/flint/domain/mapper/user/ProfileMapper.ktapp/src/main/java/com/flint/domain/model/user/UserProfileResponseModel.ktapp/src/main/java/com/flint/presentation/home/HomeScreen.ktapp/src/main/java/com/flint/presentation/home/HomeViewModel.ktapp/src/main/java/com/flint/presentation/onboarding/OnboardingUiState.ktapp/src/main/java/com/flint/presentation/setting/SettingScreen.ktapp/src/main/java/com/flint/presentation/setting/SettingUiState.ktapp/src/main/java/com/flint/presentation/setting/SettingViewModel.ktapp/src/main/java/com/flint/presentation/setting/editprofile/EditProfileUiState.ktapp/src/test/java/com/flint/data/dto/ott/OttListResponseDtoTest.ktapp/src/test/java/com/flint/presentation/onboarding/NicknameValidationTest.kt
ckals413
approved these changes
Aug 7, 2026
ckals413
left a comment
ckals413
left a comment
Contributor
There was a problem hiding this comment.
굿굿 pr이랑 이미 리뷰가 달려서 달게 없네요. 리뷰 확인만 부탁드려요!
리뷰어 섹션에 남겨주신 이메일/OTT 서버 불일치 2건은 이슈로 분리해두시면 좋을 것 같습니다!
CodeRabbit 리뷰 3건 대응 1) SettingMenuItem - 긴 trailingContent가 라벨을 밀어내던 문제 (실제 결함) trailingContent를 RowScope로 바꾸고 이메일에 weight를 준 것만으로는 효과가 없었습니다. 중첩 Row 자체는 가중치가 없어 상위 Row에서 전체 폭으로 먼저 측정되고, weight(1f)인 라벨이 남은 0에 가까운 폭을 받습니다. 라벨에서 weight를 빼 자연 폭으로 먼저 측정되게 하고, 남는 폭은 Spacer(weight)가 흡수하도록 바꿨습니다. trailingContent는 계속 우측에 붙고, 긴 이메일은 말줄임됩니다. 라벨과 붙지 않도록 최소 간격 16dp도 확보했습니다. 긴 이메일을 임시 주입해 에뮬레이터에서 확인했습니다. 2) NicknameValidationTest - 테스트 입력값이 MAX_LENGTH 초과 "플린트flint7"은 9자로 MAX_LENGTH(8)를 넘어 실제 입력 흐름에서 도달할 수 없는 값이었습니다. "플린트fl7"(6자)로 교체했습니다. 3) 홈 저장 콘텐츠 정렬 - 코드 변경 없이 계약만 명시 북마크를 순서대로 추가해 API가 최근 저장순으로 반환하는 것을 실기기에서 확인했으므로 take(10)은 올바릅니다. 다만 응답에 저장 시각 필드가 없어 클라이언트 재정렬이 불가능한 점과 서버 정렬에 의존한다는 점을 주석으로 남겼습니다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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
2 participants
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.
📮 관련 이슈
📌 작업 내용
1차 QA 시트의 로그인/회원가입 · 홈 · 설정 탭 Android fail 항목 대응입니다.
1. 닉네임 유효성 검사에 숫자 허용 (TC 1-34)
NICKNAME_REGEX에0-9가 빠져 있어 숫자가 포함된 닉네임이 거부됐습니다. 서버는 원래 숫자를 허용하고 있었고 클라이언트만 막고 있었습니다.온보딩(
OnboardingUiState)과 프로필 수정(EditProfileUiState)에 동일한 정규식이 복사돼 있어 양쪽 모두 수정했습니다. 한쪽만 고치면 "가입은flint2로 되는데 설정에서 수정은 거부"되는 불일치가 생깁니다.2. 홈 최근 저장한 콘텐츠 10개 제한 (TC 3-23, 3-24)
저장 콘텐츠가 10개를 넘어도 전부 노출되고 있었습니다.
getUserBookmarkedContents()는 프로필·저장한 콘텐츠 화면에서도 쓰이고SavedContentsSection도ProfileScreen과 공유하는 컴포넌트라, Repository나 컴포넌트가 아닌HomeViewModel에서 잘랐습니다.totalCount는 프로필 쪽에서 사용하므로 전체 개수를 유지합니다.3. 홈 OTT 바텀시트 노출 (TC 3-33, 3-34)
홈에서 저장 콘텐츠를 눌러도 바텀시트가 뜨지 않았습니다.
원인은 서버가
GET /api/v1/contents/ott/{contentId}에 대해{"otts":[]}빈 배열만 반환하는 것이었습니다. 빈 목록이면isNotEmpty()가드에 걸려 무반응이 됩니다.반면 북마크 목록 응답은 같은 콘텐츠에
getOttSimpleList를 정상적으로 내려주고 있고, 프로필·저장한 콘텐츠 화면은 이미 이 값을 사용합니다. 홈만 별도 API를 호출해서 홈에서만 실패한 구조였습니다. 홈도 동일한 방식으로 맞췄습니다.바텀시트는
OttType의 로컬iconRes/ottName으로 렌더링하므로 서버의logoUrl·contentUrl은 애초에 필요하지 않습니다.4. 설정 계정 이메일 노출 (TC 12-6, 12-7)
스웨거
MyProfileRes에email이 문서화돼 있으나UserProfileResponseDto에 필드가 없어 값이 와도 받을 수 없는 상태였습니다. DTO → 모델 → 매퍼 → UiState → UI까지 연결했습니다.email을 내려주지 않습니다. 아래 "To. 리뷰어" 참고.5. OTT 응답 DTO 방어 + 유닛 테스트
OttItemResponseDto.contentUrl이 기본값 없는 필수 필드인데 서버 스키마GetOttResponse에는 존재하지 않습니다. 지금은 서버가 빈 배열을 주고 있어 드러나지 않지만, 실제 OTT 데이터가 채워지는 순간MissingFieldException으로 깨집니다. 기본값을 지정해 막았습니다.함께 유닛 테스트 16개를 추가했습니다.
OttListResponseDtoTest—contentUrl유무 양쪽 역직렬화 (4개)NicknameValidationTest— 닉네임 숫자 허용/거부 규칙, 온보딩·프로필 수정 판정 일치 검증 (12개)📸 스크린샷
✅ 검증
Pixel_9a 에뮬레이터에서 신규 가입 후 실제 서버로 확인했습니다.
{"available":true}/contents/ott/호출 0건정렬 방향 확인: 북마크를 순서대로 추가해 API가 최신순으로 반환하는 것을 확인했습니다.
take(10)이 맞고takeLast가 아닙니다.유닛 테스트 16개 전부 통과 (
./gradlew testDebugUnitTest).😅 미구현
email을 내려주지 않아 클라이언트만으로 닫을 수 없음🫛 To. 리뷰어
백엔드 확인이 필요한 사항 2건입니다.
1.
GET /api/v1/users/me가email을 반환하지 않습니다스웨거
MyProfileRes에는"email": "이메일 (미보유 시 null)"로 문서화돼 있으나 실제 응답에는 키 자체가 없습니다.{"status":200,"data":{"id":"873202440779292987","nickname":"김종우", "isFliner":false,"keywordRecalculatable":false,"termsAgreementStatus":{...}}}카카오 로그인 시 이메일 동의항목을 수집하는지 확인이 필요합니다. 클라이언트는 값이 오면 노출, 없으면 로고만 표시하는 구조로 잡아뒀습니다.
2.
GET /api/v1/contents/ott/{contentId}가 빈 배열을 반환합니다같은 콘텐츠에 대해 북마크 API는 OTT를 정상적으로 내려주는데, 전용 OTT API는 비어 있습니다.
이번 PR은 북마크 응답 쪽 데이터를 쓰도록 우회했지만 서버 데이터 불일치는 그대로 남아 있습니다. 덧붙여
logoUrl값이 전부"adsf","asgfd","adfds"같은 더미값입니다.참고로 봐주실 부분
HomeViewModel에서ContentRepository의존성을 제거했습니다 (OTT API를 더 이상 호출하지 않음)ProfileViewModel.getOttListPerContent(142행)는 호출처가 없는 데드코드입니다. 이번 PR 범위에서 벗어나 두었는데 정리하는 게 좋을 것 같습니다SettingMenuItem의trailingContent를RowScope로 바꿨습니다. 라벨이weight(1f)라 trailing이 먼저 측정되는데, 긴 이메일이 "계정" 라벨을 밀어내는 걸 막기 위함입니다🤖 Generated with Claude Code
Summary by CodeRabbit
새로운 기능
개선
버그 수정