[Feature/#154] first qa-2 #158

leeseokchan00 · 2025-11-20T12:51:11Z

📌 PR 요약

🌱 작업한 내용

남은건 api 고치면 할게요

🌱 PR 포인트

📸 스크린샷

스크린샷
파일첨부바람

📮 관련 이슈

Resolved: [Refactor] 1차 qa 수정 2 #154

Summary by CodeRabbit

릴리스 노트

새 기능
- 하단 시트에 드래그 핸들 추가
- 등록 이미지 선택 초기화 기능 추가
- 코스명/설명 편집 시 글자수 표시 추가 (18자/20자)
UI 개선
- 상단 여백 일관성 조정(여러 화면에 상단 패딩 적용)
- 플레이스홀더 스타일 및 버튼/아이콘 크기 조정
- 일부 팝업/스낵바 표시 방식 개선
성능
- 초기 데이터 로딩 동시 실행 처리(병렬화)

_{✏️ Tip: You can customize this high-level summary in your review settings.}

coderabbitai · 2025-11-20T12:51:24Z

Walkthrough

UI 전반의 레이아웃/패딩 조정(여러 화면에 top = 50.dp 적용 및 간격 변경), 일부 버튼의 클릭 가드 제거, Dialog→Popup 전환, 이미지 선택 리셋(intent/콜백 추가), 데이터 모델 필드명/nullable 변경 및 매핑/저장소 업데이트, PlaceScreen 초기 로드의 병렬화 등이 포함된 광범위한 UI·데이터 레벨 변경입니다.

Changes

Cohort / File(s)	변경 요약
디자인 시스템 (공통 UI) `core/designsystem/src/main/java/.../bottomsheet/SolplyBasicBottomSheet.kt`, `.../button/SolplyButton.kt`, `.../card/RegisterPlaceImage.kt`, `.../card/SolplyCourseCard.kt`, `.../card/SolplyPlaceCard.kt`, `.../header/SolplyHomeHeader.kt`, `.../textfield/SolplyTextField.kt`, `.../topbar/SolplyTopBar.kt`	바텀시트에 드래그 핸들·zIndex·크기 조정, 버튼 onClick 전달 조건 변경(selected일 때만), RegisterPlaceImage에 resetSelectedUris 콜백 추가, Course/Place 카드 스타일·nullable 이미지 처리, 헤더/텍스트 필드/탑바의 스타일·패딩 수정
데이터 계층 (Place/Town DTO 및 매핑) `data/place/.../GetPlacesResponseDto.kt`, `data/place/.../mapper/PlaceEntityMapper.kt`, `data/place/.../repository/PlaceRepositoryImpl.kt`, `data/course/.../TownTreeResponseDto.kt`, `data/course/.../FavoriteTownRepositoryImpl.kt`	PlaceDto: thumbnail nullable, `mainTag`→`primaryTag`, `townId` 추가; Place 매핑·저장소와 도메인 모델에 맞춰 매핑 수정; TownNodeDto 구조 변경(parentTownId 추가·subTowns 제거) 및 getTownTree 로직 단순화
도메인 모델 `domain/place/src/main/java/.../PlaceEntity.kt`	thumbnailImageUrl nullable로 변경
원격 API 변경 `remote/place/.../PlaceService.kt`, `remote/place/.../PlaceRemoteDataSourceImpl.kt`	API 쿼리 파라미터명 변경: `subTag1Ids`→`subTagAIdList`, `subTag2Ids`→`subTagBIdList` (서버 계약 확인 필요)
검색·등록 이미지 흐름 `feature/search/.../RegisterPlaceContract.kt`, `feature/search/.../RegisterPlaceScreen.kt`, `feature/search/.../RegisterPlaceViewModel.kt`, `core/designsystem/.../RegisterPlaceImage.kt`	ResetSelectedUris intent/콜백 추가, 화면 및 뷰모델에서 인덱스 기반 선택 이미지 제거 로직 구현 및 전파
맵 관련 기능/뷰모델/UI `feature/maps/.../MapsContract.kt`, `MapsScreen.kt`, `MapsViewModel.kt`, `component/.../EditCourseBottomSheet.kt`, `RenameCourseBottomSheet.kt`, `ReportPlaceDialog.kt`, `CourseItem.kt`, `AddCourseBottomSheet.kt`, `PlaceDetailBottomSheet.kt`	Rename/ResetSelectedUris intent 추가 및 처리, RenameBottomSheet에 입력 길이 제한·문자수 표시·onChangeRenameCourse 도입, ReportPlaceDialog Dialog→Popup 전환 및 resetSelectedUris 전파, Map UI 세팅(zoom/compass 비활성화) 및 관련 콜백 연결
메인/스낵바 및 팝업 `feature/main/src/main/java/.../MainScreen.kt`	Snackbar 표시 논리를 derivedState로 분리하고 SnackbarHost를 Popup으로 래핑(비포커스·비해제 속성)
페이징/헤더·Place 관련 `feature/place/.../PlaceScreen.kt`, `feature/place/.../PlaceViewModel.kt`, `feature/place/.../model/PlaceData.kt`, `feature/place/.../PlaceOptionFilterSheet.kt`	CustomHorizontalPager에 persona/nickname 파라미터 추가(헤더 렌더링), fetchInitInfo를 병렬 async로 변경, PlaceData.thumbnailUrl nullable로 변경, 필터 초기 선택 상태 변경
온보딩·여러 화면의 클릭 가드 제거 `feature/onboarding/...`, `feature/course/.../FavoriteTownRoute.kt`, `feature/mypage/.../ProfileEditScreen.kt`, 기타 화면들 (`CollectionScreen.kt`, `MypageScreen.kt`, `WithdrawScreen.kt`, `SearchScreen.kt`, 등)	여러 화면에서 상단 패딩(top = 50.dp) 적용 및 일부 버튼의 isEnabled/조건부 가드 제거로 클릭이 항상 호출되도록 변경(버튼 스타일은 여전히 상태에 의존)
OAuth / 기타 소소한 변경 `feature/oauth/.../OauthViewModel.kt`	postSocialLogin 실패 로깅 제거

Sequence Diagram(s)

sequenceDiagram
    participant User
    participant UI as RenameCourseBottomSheet
    participant VM as MapsViewModel
    participant State as MapsState

    User->>UI: 코스명/소개 입력
    UI->>UI: 길이 검증 및 문자수 표시
    User->>UI: "완료" 클릭
    UI->>VM: onChangeRenameCourse(name,intro)
    VM->>VM: ChangeRenameCourse 처리 (courseDetailInfo 업데이트)
    VM->>UI: onStartRenameCourseClick() (편집 완료 후 흐름)

sequenceDiagram
    participant User
    participant Screen as RegisterPlaceScreen
    participant Image as RegisterPlaceImage
    participant VM as RegisterPlaceViewModel

    User->>Screen: 이미지 선택/업데이트
    Screen->>Image: selectedUris 전달
    User->>Image: 선택된 이미지 클릭 (index)
    Image->>Screen: resetSelectedUris(index)
    Screen->>VM: RegisterPlaceIntent.ResetSelectedUris(index)
    VM->>VM: 인덱스 검증 후 리스트에서 제거 및 상태 갱신
    VM->>Screen: 업데이트된 selectedUris 상태 반영

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

[Feature/#131] 제보하기 #139 — ReportPlaceDialog, RegisterPlaceImage 및 resetSelectedUris 관련 변경과 직접적으로 연결됨.
[Refactor/#144] first qa #153 — RegisterPlaceImage 및 resetSelectedUris 연결, Report/Register 경로의 공통 변화와 연관.
[Feature/#72] main place api connect #86 — PlaceDto/PlaceEntity의 thumbnail nullability 및 mainTag→primaryTag 변경과 매퍼/저장소 조정 관련.

Suggested labels

시현🥐🐟, ui🎃

Suggested reviewers

nagaeng
88guri

Poem

🐰 위로 올라간 패딩 50dp,
버튼은 더 자유롭게 폴짝!
다이얼로그는 팝업으로 깡총,
이미지 한 장 뽑아 리셋 콜,
데이터는 널도 품고 병렬로 달려요 🥕

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 3 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run `@coderabbitai generate docstrings` to improve docstring coverage.
Title check	❓ Inconclusive	PR 제목이 매우 모호하고 변경사항의 주요 내용을 명확하게 전달하지 못함. '#154' 이슈 번호와 'first qa-2'는 구체적인 변경사항을 설명하지 않음.	PR 제목을 더 구체적으로 변경하여 주요 변경사항을 명확하게 표현하세요. 예: 'Design system UI 개선 및 API 매핑 변경' 또는 '디자인 시스템 UI 조정 및 데이터 모델 업데이트'.
Linked Issues check	❓ Inconclusive	연결된 이슈 #154는 체크리스트만 포함하고 구체적인 요구사항이 명시되지 않음. 코딩 관련 요구사항을 검증할 수 없음.	이슈 #154에 구체적인 코딩 요구사항과 목표를 추가하여 PR의 변경사항과 매핑할 수 있도록 명시하세요.
Out of Scope Changes check	❓ Inconclusive	이슈에 구체적인 요구사항이 없어 범위 판단이 어려우나, 변경사항들이 디자인 시스템 UI, 데이터 모델, 화면 레이아웃 조정으로 일관성 있게 보임.	이슈 #154의 구체적인 요구사항이 명시되어야 범위 내/외의 변경사항을 명확하게 판단할 수 있습니다.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

✨ Finishing touches

📝 Generate docstrings

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.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

coderabbitai

Actionable comments posted: 4

🧹 Nitpick comments (10)

feature/maps/src/main/java/com/teamsolply/solply/maps/component/CourseItem.kt (3)
108-110: 타이틀 1줄 제한 시 Ellipsis 명시를 고려

타이틀을 maxLines = 1로 제한하면서 overflow 정책이 기본값(Clip)에 의존하고 있습니다. 디자인 상 잘린다는 걸 사용자에게 명확히 보여주고 싶다면 TextOverflow.Ellipsis를 명시하는 편이 더 직관적일 수 있습니다.
-                            Text(
-                                text = placeName.formatTextToPlaceItemTitle(),
-                                modifier = Modifier,
-                                color = SolplyTheme.colors.black,
-                                style = SolplyTheme.typography.title15M,
-                                maxLines = 1
-                            )
+                            Text(
+                                text = placeName.formatTextToPlaceItemTitle(),
+                                modifier = Modifier,
+                                color = SolplyTheme.colors.black,
+                                style = SolplyTheme.typography.title15M,
+                                maxLines = 1,
+                                overflow = TextOverflow.Ellipsis
+                            )
113-123: 주소 Text의 maxLines/overflow 정책 일관성 개선 제안

선택 상태에서만 maxLines = 2 + Ellipsis, 비선택 상태에서는 maxLines = 1 + Clip 조합인데, 비선택 상태에서도 길이가 약간만 길어도 마지막 글자가 딱 잘려 보일 수 있습니다. UX 관점에서 항상 말줄임표를 사용하는 쪽이 자연스러울 수도 있습니다.

필요하다면 다음처럼 overflow 정책만 단순화하는 것도 고려해볼 만합니다.
-                            style = SolplyTheme.typography.caption12R,
-                            maxLines = if (selectedPlaceItem) 2 else 1,
-                            overflow = if (selectedPlaceItem) {
-                                TextOverflow.Ellipsis
-                            } else {
-                                TextOverflow.Clip
-                            }
+                            style = SolplyTheme.typography.caption12R,
+                            maxLines = if (selectedPlaceItem) 2 else 1,
+                            overflow = TextOverflow.Ellipsis
125-145: Spacer 폭 명시로 아이콘 영역 여백 표현이 더 명확해짐

Spacer(modifier = Modifier.width(12.dp))로 텍스트와 아이콘 사이 간격을 고정값으로 둔 선택은 직관적이고, Row 내 구성 요소 간 여백 의도가 잘 드러납니다.

한 가지 선택적 제안으로, 마지막 Spacer(modifier = Modifier.padding(end = 22.dp))도 아래처럼 width 기반으로 통일하면 레이아웃 의도를 읽는 입장에서 조금 더 명확할 수 있습니다.
-                    Spacer(modifier = Modifier.padding(end = 22.dp))
+                    Spacer(modifier = Modifier.width(22.dp))
core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/header/SolplyHomeHeader.kt (1)
32-33: 헤더 여백 대폭 증가

상단 패딩이 11.dp에서 48.dp로 크게 증가했습니다. 다른 화면들은 대부분 50.dp를 사용하는데, 여기서는 48.dp를 사용하는 이유가 있나요? 의도적인 차이가 아니라면 일관성을 위해 50.dp 사용을 고려해보세요.

일관성을 위해 다음과 같이 수정을 고려해보세요:
         .fillMaxWidth()
         .background(color = SolplyTheme.colors.white)
-        .padding(top = 48.dp)
+        .padding(top = 50.dp)
feature/place/src/main/java/com/teamsolply/solply/place/component/bottomsheet/PlaceOptionFilterSheet.kt (1)
95-95: 초기화 버튼의 선택 상태를 재고하세요.

"초기화" 버튼의 selected 속성이 항상 true로 설정되어 있습니다. 선택된 필터가 있을 때만 버튼이 활성화 상태로 표시되어야 사용자에게 더 명확한 피드백을 제공할 수 있습니다.

다음과 같이 수정을 고려하세요:
 SolplyBasicButton(
     text = "초기화",
     onClick = onReset,
     modifier = Modifier.weight(1f),
-    selected = true,
+    selected = selectedOptionIds.isNotEmpty(),
     textColor = SolplyTheme.colors.gray900,
     textStyle = SolplyTheme.typography.body16M,
     enabledBackgroundColor = SolplyTheme.colors.white,
     disabledBackgroundColor = SolplyTheme.colors.white
 )
feature/place/src/main/java/com/teamsolply/solply/place/PlaceViewModel.kt (1)
152-183: fetchInitInfo의 async/await 사용 구조 단순화/명확화 제안

fetchPlaces, fetchRecommendPlace, fetchMainTags 각각이 내부에서 이미 viewModelScope.launch { ... }를 사용하고 있어서, 여기서 async { fetchXxx() }로 감싼 뒤 await()를 호출해도 실제 네트워크 작업 완료를 기다리지는 않고, 기존(단순 함수 호출)과 동작이 거의 같습니다.

의도가 단순히 세 작업을 동시에 시작하는 것이라면, 아래처럼 원래처럼 단순 호출로 두어도 되고, 구조적 동시성을 명확히 가져가고 싶다면 세 함수를 suspend로 바꾼 뒤 이 스코프 안에서만 async/await를 사용하는 방향도 고려해 볼 만합니다.
// 간단히 동일 동작을 유지할 경우
fetchPlaces(...)
fetchRecommendPlace(...)
fetchMainTags()
큰 버그는 아니지만, 나중에 await()가 실제 완료를 보장한다고 오해할 수 있어서 한 번 검토해 보시면 좋겠습니다.
feature/search/src/main/java/com/teamsolply/solply/registerplace/RegisterPlaceContract.kt (1)

36-55: ResetSelectedUris의 index 의미를 조금 더 명확히 표현하면 좋겠습니다

새 인텐트 추가 방향 자체는 좋습니다. 다만 index가 0 기반 인덱스인지, 어떤 리스트(예: selectedReportUris)를 대상으로 하는지 코드만으로는 약간 모호해서,

이름을 ResetSelectedReportUri 정도로 더 구체적으로 바꾸거나

KDoc/주석으로 index 의미를 설명

해두면 호출 측에서 실수할 여지가 더 줄어들 것 같습니다.
feature/search/src/main/java/com/teamsolply/solply/registerplace/RegisterPlaceViewModel.kt (1)
106-131: 선택 이미지 리셋 로직은 안전하지만, 리스트 타입 일관성은 조금 더 맞출 수 있습니다

target.isEmpty() 및 intent.index !in target.indices 체크로 방어 로직이 잘 들어가 있어서 기능적으로는 안전해 보입니다.

다만 RegisterPlaceState에서는 기본적으로 persistentListOf()를 사용하고 있는데, 여기서는 toMutableList() 후 MutableList를 그대로 상태에 넣고 있어 컬렉션 타입 일관성이 약간 깨질 수 있습니다. 아래처럼 다시 immutable 리스트로 바꿔 넣는 쪽을 한 번 고려해 보셔도 좋겠습니다.
is RegisterPlaceIntent.ResetSelectedUris -> reduce {
    val target = selectedReportUris.toMutableList()
    if (target.isEmpty() || intent.index !in target.indices) return@reduce this
    target.removeAt(intent.index)
    copy(selectedReportUris = target.toPersistentList())
}
(필요 시 kotlinx.collections.immutable.toPersistentList import 추가)
feature/place/src/main/java/com/teamsolply/solply/place/PlaceScreen.kt (1)

488-496: Persona 매핑 로직

getRecommendText 함수가 persona 타입에 따라 다른 추천 메시지를 반환합니다. 현재는 4가지 persona 타입과 fallback을 처리하고 있습니다. 향후 persona 타입이 추가되거나 다른 곳에서도 사용될 경우, 중앙 집중식 persona 관리 유틸리티로 리팩토링하는 것을 고려해볼 수 있습니다.
data/course/src/main/java/com/teamsolply/solply/course/favoriteTown/repository/FavoriteTownRepositoryImpl.kt (1)
15-38: 데이터 변환 로직 리팩토링

중첩된 트리 구조 대신 평면 리스트를 필터링하여 지역과 타운을 구성하는 방식으로 개선되었습니다. 로직이 더 명확하고 이해하기 쉽습니다.

선택적 개선사항: Line 28의 groupBy { it.parentTownId ?: 0L }에서 ?: 0L 부분은 불필요합니다. Line 27의 필터 조건에서 이미 null과 0을 제외했기 때문입니다.
-        val townsByRegion: Map<Long, List<TownLite>> = towns
-            .filter { it.parentTownId != null && it.parentTownId != 0L }
-            .groupBy { it.parentTownId ?: 0L }
+        val townsByRegion: Map<Long, List<TownLite>> = towns
+            .filter { it.parentTownId != null && it.parentTownId != 0L }
+            .groupBy { it.parentTownId!! }

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between db6473b and ac7ced2.

📒 Files selected for processing (48)

core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/bottomsheet/SolplyBasicBottomSheet.kt (2 hunks)
core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/button/SolplyButton.kt (1 hunks)
core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/card/RegisterPlaceImage.kt (2 hunks)
core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/card/SolplyCourseCard.kt (1 hunks)
core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/card/SolplyPlaceCard.kt (2 hunks)
core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/header/SolplyHomeHeader.kt (2 hunks)
core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/textfield/SolplyTextField.kt (1 hunks)
core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/topbar/SolplyTopBar.kt (1 hunks)
data/course/src/main/java/com/teamsolply/solply/course/favoriteTown/dto/TownTreeResponseDto.kt (1 hunks)
data/course/src/main/java/com/teamsolply/solply/course/favoriteTown/repository/FavoriteTownRepositoryImpl.kt (1 hunks)
data/place/src/main/java/com/teamsolply/solply/place/dto/response/GetPlacesResponseDto.kt (1 hunks)
data/place/src/main/java/com/teamsolply/solply/place/mapper/PlaceEntityMapper.kt (1 hunks)
data/place/src/main/java/com/teamsolply/solply/place/repository/PlaceRepositoryImpl.kt (1 hunks)
domain/place/src/main/java/com/teamsolply/solply/place/model/PlaceEntity.kt (1 hunks)
feature/collection/src/main/java/com/teamsolply/solply/collection/CollectionMenuScreen.kt (1 hunks)
feature/collection/src/main/java/com/teamsolply/solply/collection/collection/component/CollectionScreen.kt (2 hunks)
feature/course/src/main/java/com/teamsolply/solply/course/component/FavoriteTownTopBar.kt (1 hunks)
feature/course/src/main/java/com/teamsolply/solply/course/favoriteTown/FavoriteTownRoute.kt (1 hunks)
feature/main/src/main/java/com/teamsolply/solply/main/MainScreen.kt (16 hunks)
feature/maps/src/main/java/com/teamsolply/solply/maps/MapsContract.kt (1 hunks)
feature/maps/src/main/java/com/teamsolply/solply/maps/MapsScreen.kt (7 hunks)
feature/maps/src/main/java/com/teamsolply/solply/maps/MapsViewModel.kt (2 hunks)
feature/maps/src/main/java/com/teamsolply/solply/maps/component/CourseItem.kt (4 hunks)
feature/maps/src/main/java/com/teamsolply/solply/maps/component/bottomsheet/AddCourseBottomSheet.kt (1 hunks)
feature/maps/src/main/java/com/teamsolply/solply/maps/component/bottomsheet/EditCourseBottomSheet.kt (3 hunks)
feature/maps/src/main/java/com/teamsolply/solply/maps/component/bottomsheet/PlaceDetailBottomSheet.kt (1 hunks)
feature/maps/src/main/java/com/teamsolply/solply/maps/component/bottomsheet/RenameCourseBottomSheet.kt (7 hunks)
feature/maps/src/main/java/com/teamsolply/solply/maps/component/dialog/ReportPlaceDialog.kt (9 hunks)
feature/mypage/src/main/java/com/teamsolply/solply/mypage/MypageScreen.kt (2 hunks)
feature/mypage/src/main/java/com/teamsolply/solply/mypage/component/MypagePlaceAllScreen.kt (1 hunks)
feature/mypage/src/main/java/com/teamsolply/solply/mypage/profile/ProfileEditScreen.kt (2 hunks)
feature/mypage/src/main/java/com/teamsolply/solply/mypage/withdraw/WithdrawScreen.kt (1 hunks)
feature/oauth/src/main/java/com/teamsolply/solply/oauth/OauthViewModel.kt (0 hunks)
feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/OnBoardingScreen.kt (1 hunks)
feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/screen/AllowClauseScreen.kt (1 hunks)
feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/screen/NamingScreen.kt (1 hunks)
feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/screen/SelectPersonaScreen.kt (1 hunks)
feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/screen/SelectTownScreen.kt (2 hunks)
feature/place/src/main/java/com/teamsolply/solply/place/PlaceScreen.kt (5 hunks)
feature/place/src/main/java/com/teamsolply/solply/place/PlaceViewModel.kt (2 hunks)
feature/place/src/main/java/com/teamsolply/solply/place/component/bottomsheet/PlaceOptionFilterSheet.kt (1 hunks)
feature/place/src/main/java/com/teamsolply/solply/place/model/PlaceData.kt (1 hunks)
feature/search/src/main/java/com/teamsolply/solply/registerplace/RegisterPlaceContract.kt (1 hunks)
feature/search/src/main/java/com/teamsolply/solply/registerplace/RegisterPlaceScreen.kt (6 hunks)
feature/search/src/main/java/com/teamsolply/solply/registerplace/RegisterPlaceViewModel.kt (1 hunks)
feature/search/src/main/java/com/teamsolply/solply/search/SearchScreen.kt (2 hunks)
remote/place/src/main/java/com/teamsolply/solply/place/datasource/PlaceRemoteDataSourceImpl.kt (1 hunks)
remote/place/src/main/java/com/teamsolply/solply/place/service/PlaceService.kt (1 hunks)

💤 Files with no reviewable changes (1)

feature/oauth/src/main/java/com/teamsolply/solply/oauth/OauthViewModel.kt

🧰 Additional context used

🧬 Code graph analysis (8)

feature/search/src/main/java/com/teamsolply/solply/registerplace/RegisterPlaceViewModel.kt (1)

core/ui/src/main/java/com/teamsolply/solply/ui/base/BaseViewModel.kt (1)

reduce (44-46)

feature/maps/src/main/java/com/teamsolply/solply/maps/component/bottomsheet/RenameCourseBottomSheet.kt (1)

core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/button/SolplyButton.kt (1)

SolplyBasicButton (57-83)

feature/maps/src/main/java/com/teamsolply/solply/maps/MapsViewModel.kt (1)

core/ui/src/main/java/com/teamsolply/solply/ui/base/BaseViewModel.kt (1)

reduce (44-46)

feature/maps/src/main/java/com/teamsolply/solply/maps/component/dialog/ReportPlaceDialog.kt (1)

core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/button/SolplyButton.kt (1)

SolplyBasicButton (57-83)

feature/place/src/main/java/com/teamsolply/solply/place/PlaceScreen.kt (1)

core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/header/SolplyHomeHeader.kt (1)

SolplyHomeHeader (22-82)

core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/card/SolplyPlaceCard.kt (1)

core/ui/src/main/java/com/teamsolply/solply/ui/image/AdaptationImage.kt (1)

AdaptationImage (15-69)

feature/main/src/main/java/com/teamsolply/solply/main/MainScreen.kt (3)

feature/course/src/main/java/com/teamsolply/solply/course/favoriteTown/favoriteTownNavigation/FavoriteTownNavigation.kt (1)

favoriteTownNavGraph (20-32)

feature/search/src/main/java/com/teamsolply/solply/search/navigation/SearchNavigation.kt (1)

searchNavGraph (16-30)

core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/snackbar/SolplySnackBar.kt (4)

SolplyTextSnackBar (37-53)

SolplyNotificationSnackBar (55-78)

SolplyNavigateSnackBar (80-113)

SolplyNavigateSimpleSnackBar (115-147)

feature/search/src/main/java/com/teamsolply/solply/search/SearchScreen.kt (1)

feature/main/src/main/java/com/teamsolply/solply/main/MainNavigator.kt (1)

navigateToBack (116-118)

🔇 Additional comments (70)

feature/maps/src/main/java/com/teamsolply/solply/maps/component/bottomsheet/AddCourseBottomSheet.kt (1)

63-67: 28.dp 터치 영역 크기 확인 필요 - 설계 의도 검증

코드 변경 사항을 검증한 결과, 28.dp 크기는 이 프로젝트에서 보조 작업 아이콘(secondary action icons)의 일관된 패턴입니다:

EditCourseBottomSheet.kt 줄 159: 편집 아이콘 28.dp

CourseItem.kt 줄 130: 편집 아이콘 28.dp

다른 보조 작업 아이콘들: 20.dp, 24.dp

주요 작업 버튼만: 48.dp (MainBottomBar)

28.dp가 Android 권장 48.dp보다 작은 것은 사실이지만, 현재 프로젝트의 설계 패턴입니다. rippleEnabled = false 설정은 시각적 일관성을 유지하려는 의도로 보입니다. 프로젝트 설계 시스템 문서에서 이 크기 선택이 의도된 것인지 확인해 주시기 바랍니다.

feature/maps/src/main/java/com/teamsolply/solply/maps/component/CourseItem.kt (3)

32-32: TextOverflow import 추가 적절

아래 Text 컴포저블에서 overflow 설정을 위해 필요한 import이고, 불필요한 의존성도 없어 그대로 유지해도 좋습니다.

65-69: 선택 상태별 주소 포맷 분리 로직 적절

selectedPlaceItem 여부에 따라 원본 주소 vs 축약/포맷팅 주소를 분리해서 사용하는 구조가 명확하고, 가독성도 좋아졌습니다. 동일 로직을 여러 곳에서 중복 호출하지 않는 점도 좋습니다.

98-98: Row 내 텍스트 영역에 weight 부여한 것 적절

Column(modifier = Modifier.weight(1f))로 텍스트 영역이 가변 폭을 가지도록 하고, 오른쪽 아이콘 영역을 고정 폭으로 두는 구조가 레이아웃 안정성에 도움이 됩니다. 특히 긴 타이틀/주소가 들어올 때 아이콘이 밀리지 않도록 하는 데 유효해 보입니다.

feature/course/src/main/java/com/teamsolply/solply/course/component/FavoriteTownTopBar.kt (1)

29-29: 레이아웃 간격 조정이 적절합니다.

상단 패딩을 50.dp로 증가시켜 전체 앱의 일관된 간격 정책을 따르고 있습니다.

feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/OnBoardingScreen.kt (1)

118-118: 상단 간격 조정이 일관성 있게 적용되었습니다.

다른 화면들과 동일하게 50.dp 간격을 적용하여 일관된 UI를 제공합니다.

feature/mypage/src/main/java/com/teamsolply/solply/mypage/withdraw/WithdrawScreen.kt (1)

105-106: 레이아웃 패딩이 일관되게 적용되었습니다.

배경 적용 후 상단 패딩 50.dp를 추가하여 다른 화면들과 동일한 간격을 유지합니다.

feature/maps/src/main/java/com/teamsolply/solply/maps/component/bottomsheet/PlaceDetailBottomSheet.kt (1)

392-392: 바텀시트 내부 간격 조정이 적절합니다.

SNS 링크 섹션과 오류 제보 섹션 사이의 간격을 20.dp로 증가시켜 가독성을 개선했습니다.

feature/collection/src/main/java/com/teamsolply/solply/collection/CollectionMenuScreen.kt (1)

136-137: 일관된 상단 간격이 적용되었습니다.

다른 메인 화면들과 동일하게 50.dp 상단 패딩을 적용하여 통일된 레이아웃을 구성합니다.

feature/mypage/src/main/java/com/teamsolply/solply/mypage/component/MypagePlaceAllScreen.kt (1)

38-39: 레이아웃 간격이 일관되게 적용되었습니다.

배경 적용 후 상단 패딩 50.dp를 추가하여 앱 전체의 일관된 간격 정책을 따릅니다.

core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/topbar/SolplyTopBar.kt (1)

48-48: 텍스트 패딩 제거가 적절합니다.

컴포넌트 내부 패딩을 제거하고 상위 컨테이너에서 일관된 간격을 관리하는 방식으로 변경되었습니다. 이는 다른 파일들에서 상단 패딩을 추가한 것과 일관성을 유지합니다.

feature/course/src/main/java/com/teamsolply/solply/course/favoriteTown/FavoriteTownRoute.kt (1)

103-103: ViewModel의 selectedTownId 검증이 이미 존재하여 안전성 확보됨

검증 결과, FavoriteTownRoute.kt 라인 103의 onClick 핸들러는 isButtonEnabled 상태와 관계없이 ConfirmSelection 인텐트를 전송합니다. 하지만 FavoriteViewModel.patchUserFavoriteTown() (라인 47)에서 val selectedTownId = s.selectedTownId ?: return로 null을 체크하고 조기 반환하므로, 버튼이 시각적으로 비활성화된 상태에서 클릭되어도 실제 작업은 수행되지 않습니다. ViewModel이 상태를 검증하므로 현재 구현은 안전합니다.

feature/mypage/src/main/java/com/teamsolply/solply/mypage/profile/ProfileEditScreen.kt (2)

120-121: 상단 여백 추가 확인

다른 화면들과 일관되게 50.dp 상단 패딩이 추가되었습니다. PR 전반에 걸친 UI 간격 표준화 작업으로 보입니다.

201-201: Now I need to examine the SolplyButton.kt file directly to understand the implementation:The web search results aren't specific to the codebase. Based on the earlier findings, I have enough information to make a determination. Let me now examine the specific usages more carefully and provide a comprehensive analysis:

Based on my analysis of the script results showing all SolplyBasicButton usages, I can now provide a comprehensive assessment:

검증 로직 이동에 따른 button 동작 변경 확인 완료

코드베이스 전체에서 SolplyBasicButton의 사용 패턴을 분석한 결과, 다음과 같은 불일치가 확인됩니다:

현재 코드베이스 패턴:

WithdrawScreen.kt (162-167): if (buttonEnabled) 조건 체크를 onClick 내부에서 수행

ProfileEditScreen.kt (변경 후): selected = isEditSuccess이지만 onClick 무조건 호출

나머지 화면들 (SelectTownScreen, SelectPersonaScreen, AllowClauseScreen 등): selected = isButtonEnabled 등의 파라미터 사용

문제점:
다음 두 가지 패턴이 혼재되어 있습니다:

검증 로직을 호출자에서 처리 (WithdrawScreen 패턴)

검증을 selected 파라미터로 처리 (변경된 ProfileEditScreen 패턴)

이 변경사항이 의도된 동작인지 확인이 필요하며, SolplyBasicButton의 selected=false일 때 onClick 호출 여부를 명확히 해야 합니다. 다른 화면들(특히 RegisterPlaceScreen에서 복잡한 선택 조건 사용)과의 일관성을 위해 컴포넌트의 동작 방식을 문서화하는 것을 권장합니다.

feature/collection/src/main/java/com/teamsolply/solply/collection/collection/component/CollectionScreen.kt (1)

46-47: 일관된 상단 여백 적용

다른 화면들과 동일하게 50.dp 상단 패딩이 추가되었습니다. 전체 앱의 UI 간격 일관성을 위한 변경으로 보입니다.

feature/mypage/src/main/java/com/teamsolply/solply/mypage/MypageScreen.kt (3)

128-129: 일관된 레이아웃 간격 적용

다른 화면들과 동일하게 50.dp 상단 패딩이 추가되었습니다. 배경색도 명시적으로 설정되어 레이아웃이 명확해졌습니다.

134-134: Modifier 체이닝 간소화

Row의 modifier가 단일 호출로 간소화되었습니다. 코드가 더 깔끔해졌습니다.

143-143: 뒤로 가기 버튼 패딩 조정

상단 여백이 Column 레벨에서 적용되면서 Icon의 개별 top padding이 제거되었습니다. 새로운 레이아웃 구조와 잘 맞습니다.

core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/textfield/SolplyTextField.kt (1)

398-399: 플레이스홀더 스타일링 개선

플레이스홀더의 색상이 더 진하게(gray500 → gray600) 변경되고, 텍스트 스타일이 더 크고 읽기 쉬운 형태(caption12M → body14R)로 변경되었습니다. 가독성이 개선되는 변경사항입니다.

core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/card/SolplyCourseCard.kt (1)

82-86: 오버레이 모서리 둥글기 개선

반투명 오버레이에 상단 모서리 둥글기(4.dp)가 추가되었습니다. 이제 모든 모서리가 둥글게 처리되어 카드의 시각적 완성도가 높아졌습니다.

feature/search/src/main/java/com/teamsolply/solply/search/SearchScreen.kt (2)

99-99: 일관된 상단 여백 적용

다른 화면들과 동일하게 상단 패딩이 50.dp로 증가되었습니다. PR 전반의 UI 간격 표준화 작업입니다.

103-107: 터치 영역 명확화

클릭 가능 영역이 Row 전체에서 Icon으로 변경되었습니다. 사용자 인터랙션 영역이 더 명확해졌습니다.

core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/button/SolplyButton.kt (1)

72-72: onClick 조건부 실행 - 설계 일관성 및 문서화 개선 필요

현재 코드베이스의 모든 사용처(SelectPersonaScreen, SelectTownScreen, AllowClauseScreen, ProfileEditScreen, FavoriteTownRoute 등 8개 파일)를 검토한 결과, selected 파라미터는 이미 enabledBackgroundColor/disabledBackgroundColor와 함께 enable/disable 상태를 제어하는 의도된 패턴으로 사용되고 있습니다.

다만, 다음 개선사항이 필요합니다:

설계 불일치: 같은 파일의 AddLocalAreaButton은 selected 상태와 관계없이 onClick을 항상 실행합니다. SolplyBasicButton과의 동작 일관성을 검토하세요.

명확한 문서화: selected=false일 때 onClick이 호출되지 않는 동작을 KDoc 주석으로 명시하여 사용자가 API 계약을 명확히 이해할 수 있도록 하세요.

대안 검토: enabled 파라미터를 명시적으로 추가하거나, 함수 이름/주석을 통해 이 동작이 의도적임을 분명히 하는 것이 좋습니다.

feature/maps/src/main/java/com/teamsolply/solply/maps/MapsContract.kt (1)

88-89: 새로운 인텐트 추가가 적절합니다.

ResetSelectedUris와 ChangeRenameCourse 인텐트가 기존 패턴과 일관되게 추가되었으며, 파라미터 타입도 적절합니다.

core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/bottomsheet/SolplyBasicBottomSheet.kt (4)

33-35: 바텀 시트 크기 조정을 확인하세요.

fullyExpanded와 slightlyExpanded 값이 변경되었습니다. 다양한 화면 크기와 디바이스에서 바텀 시트가 의도한 대로 표시되는지 확인하세요.

40-40: zIndex 레이어링의 영향을 검증하세요.

zIndex(1f)가 추가되어 바텀 시트가 다른 UI 요소 위에 표시됩니다. 다른 팝업이나 다이얼로그와의 z-order 충돌이 없는지 확인하세요.

46-46: 시스템 인셋 처리를 확인하세요.

WindowInsets(0, 0, 0, 0)로 명시적으로 설정하면 노치나 둥근 모서리가 있는 기기에서 시스템 바 영역이 올바르게 처리되지 않을 수 있습니다.

47-57: 드래그 핸들 추가가 좋습니다.

사용자 경험을 개선하는 시각적 드래그 핸들이 추가되었습니다.

feature/main/src/main/java/com/teamsolply/solply/main/MainScreen.kt (3)

123-128: 변수명 변경이 일관성을 개선합니다.

innerPadding을 paddingValue로 일관되게 변경하여 코드 가독성이 향상되었습니다.

437-481: Popup의 해제 불가 동작을 검증하세요.

스낵바가 Popup으로 래핑되어 dismissOnBackPress = false와 dismissOnClickOutside = false로 설정되어 있습니다. 이는 사용자가 스낵바를 수동으로 해제할 수 없음을 의미합니다. 모든 스낵바 타입에서 이것이 의도된 동작인지 확인하세요.

440-440: 하드코딩된 오프셋 값을 확인하세요.

오프셋이 IntOffset(0, -32)로 하드코딩되어 있습니다. 다양한 화면 크기와 방향에서 스낵바 위치가 적절한지 확인하세요.
core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/card/SolplyPlaceCard.kt (2)
87-100: null 이미지 처리에 대한 동작을 명확히 하세요.

imgRes가 null일 때 AdaptationImage가 렌더링되지 않으며, 결과적으로 isImageReady가 항상 false로 유지됩니다. 이는 이미지가 없는 카드를 클릭할 수 없게 만듭니다 (line 64의 touchable && isImageReady 조건). 이것이 의도된 동작인지 확인하세요. 이미지가 없는 장소 카드도 클릭 가능해야 할 수 있습니다.

이미지가 없어도 카드를 클릭 가능하게 하려면:
 Column(
     modifier = modifier
         .fillMaxWidth()
         .then(
-            if (touchable && isImageReady) {
+            if (touchable && (isImageReady || imgRes == null)) {
                 Modifier.customClickable(rippleEnabled = false) { onClick() }
             } else {
                 Modifier
             }
         ),
42-42: 검토 의견이 부정확합니다 - Kotlin 타입 시스템에 대한 오해입니다.

imgRes 파라미터를 String에서 String?로 변경하는 것은 Kotlin에서 breaking change가 아닙니다. Kotlin의 타입 호환성 규칙에 따르면, 매개변수에 null 가능성을 추가하는 것은 **역방향 호환(backward compatible)**입니다. 기존 호출자는 String 값을 계속 String? 매개변수에 전달할 수 있으므로 수정할 필요가 없습니다.

Breaking change가 되려면 반대 방향이어야 합니다 (String? → String은 모든 호출자가 null 처리를 추가해야 함).

실제로 확인된 사항:

함수 구현에서 if (imgRes != null) 조건으로 null을 올바르게 처리

4개 호출 위치 모두 호환 가능한 값 전달

호출자 업데이트 불필요

Likely an incorrect or invalid review comment.
data/place/src/main/java/com/teamsolply/solply/place/repository/PlaceRepositoryImpl.kt (1)

108-108: 필드 이름 변경이 일관적으로 적용되었습니다.

dto.mainTag에서 dto.primaryTag로 변경이 매퍼와 일관되게 적용되었습니다.

data/place/src/main/java/com/teamsolply/solply/place/mapper/PlaceEntityMapper.kt (1)

11-11: 검증 완료: 필드 매핑이 정확합니다.

PlaceDto에 primaryTag 필드가 존재하며(data/place/src/main/java/com/teamsolply/solply/place/dto/response/GetPlacesResponseDto.kt 24번 줄), PlaceEntityMapper.kt 11번 줄의 매핑이 올바르게 적용되었습니다. mainTag에서 primaryTag로의 필드명 변경이 제대로 반영되어 있습니다.

feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/screen/SelectPersonaScreen.kt (1)

67-75: 선택되지 않은 상태에서도 onNextClick이 호출되는 동작 의도 확인 필요

이제 isButtonEnabled가 false여도 onNextClick()이 항상 실행됩니다. SolplyBasicButton이 자체적으로 비활성화 처리를 하지 않는다면, 페르소나 미선택 상태에서도 온보딩 다음 단계로 진행될 수 있어서, 이 케이스를 onNextClick 쪽에서 검증/차단하고 있는지 한 번만 확인 부탁드립니다.

feature/place/src/main/java/com/teamsolply/solply/place/PlaceViewModel.kt (1)

12-12: async 도입을 위한 import 추가는 적절해 보입니다

아래 fetchInitInfo에서 병렬 호출을 위해 async를 사용하는 구조와 일관된 import라서 문제 없어 보입니다.

domain/place/src/main/java/com/teamsolply/solply/place/model/PlaceEntity.kt (1)

3-9: thumbnailImageUrl nullable 전환에 따른 사용처 null 처리 확인 권장

thumbnailImageUrl가 String?로 변경되면서, 이를 사용하는 매퍼/UI 쪽에서도 모두 null 허용 전제로 바뀌어야 합니다. 요약상 PlaceData.thumbnailUrl도 nullable로 맞춰진 것 같지만,

!!.,

바로 이미지 로더에 넘기면서 null을 고려하지 않는 코드,

가 남아 있으면 런타임 NPE나 플레이스홀더 미표시 문제가 생길 수 있으니, 해당 필드 사용처 한 번만 전체적으로 스캔해 봐 주시면 좋겠습니다.

feature/place/src/main/java/com/teamsolply/solply/place/model/PlaceData.kt (1)

8-8: nullable 변경 사항 확인 완료

thumbnailUrl을 nullable로 변경한 것은 API 응답에서 썸네일이 없는 경우를 처리하기 위한 적절한 변경입니다. 관련된 PlaceEntity, PlaceDto 등 데이터 레이어 전반에 걸쳐 일관되게 적용되었습니다.

feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/screen/NamingScreen.kt (1)

69-70: 조건부 가드 제거 확인

onClick 핸들러에서 state.isOnBoardingSuccess 체크를 제거하고 무조건 ShowStartingScreen을 호출하도록 변경되었습니다. AI summary에 따르면 SolplyBasicButton이 selected 상태일 때만 내부적으로 onClick을 전달하므로, 가드 로직이 컴포넌트 레벨로 이동한 것으로 보입니다.

feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/screen/SelectTownScreen.kt (2)

141-141: 조건부 가드 제거 - 일관된 패턴

"완료" 버튼의 onClick 핸들러에서 isButtonEnabled 체크를 제거했습니다. SolplyBasicButton이 selected 상태를 기반으로 내부적으로 클릭을 처리하므로, NamingScreen 및 다른 온보딩 화면들과 일관된 패턴입니다.

157-157: 조건부 가드 제거 확인

"다음" 버튼도 동일하게 조건부 가드가 제거되었으며, 버튼 컴포넌트 레벨에서 상태 관리가 이루어집니다.

remote/place/src/main/java/com/teamsolply/solply/place/datasource/PlaceRemoteDataSourceImpl.kt (1)

37-50: API 파라미터 이름 변경 확인

subTag1Ids/subTag2Ids를 subTagAIdList/subTagBIdList로 변경했습니다. null 체크 및 쉼표 구분 문자열 변환 로직은 그대로 유지되어 기능상 문제가 없습니다. PlaceService의 쿼리 파라미터 이름과도 일관되게 정렬되었습니다.

remote/place/src/main/java/com/teamsolply/solply/place/service/PlaceService.kt (1)

32-33: Retrofit 쿼리 파라미터 이름 변경

API 계약에 맞춰 쿼리 파라미터 이름을 subTag1Ids/subTag2Ids에서 subTagAIdList/subTagBIdList로 변경했습니다. 파라미터 타입은 동일하게 유지되어 있습니다.

feature/maps/src/main/java/com/teamsolply/solply/maps/component/bottomsheet/EditCourseBottomSheet.kt (4)

83-84: 새 콜백 파라미터 추가

코스 편집 완료를 처리하기 위한 finishEditCourseClick 파라미터가 추가되었습니다. MapsViewModel의 새로운 intent와 함께 코스 편집 플로우를 개선하는 변경사항입니다.

151-151: 간격 조정

코스 이름 하단 패딩이 4.dp에서 8.dp로 증가했습니다. 시각적 간격 개선을 위한 마이너 UI 조정입니다.

158-162: 편집 아이콘 클릭 가능 영역 개선

편집 아이콘에 customClickable을 적용하고 크기를 24.dp에서 28.dp로 증가시켜 터치 영역이 개선되었습니다. 사용자 경험 향상에 도움이 됩니다.

300-300: 완료 버튼 동작 변경

"완료" 버튼이 이제 finishEditCourseClick()을 호출합니다. 새로운 콜백 구조와 일관된 변경입니다.

core/designsystem/src/main/java/com/teamsolply/solply/designsystem/component/card/RegisterPlaceImage.kt (2)

37-37: 이미지 선택 해제 기능 추가

resetSelectedUris 파라미터를 추가하여 선택된 이미지를 탭하여 제거할 수 있는 기능이 구현되었습니다. 사용자 경험 개선에 도움이 되는 변경입니다.

51-54: 선택된 이미지 클릭 처리

선택된 이미지에 customClickable을 적용하여 클릭 시 resetSelectedUris(index)를 호출하도록 구현되었습니다. 직관적인 이미지 제거 UX를 제공합니다.

feature/place/src/main/java/com/teamsolply/solply/place/PlaceScreen.kt (2)

234-235: 개인화된 추천 텍스트 기능 추가

CustomHorizontalPager에 persona와 nickname 파라미터를 전달하여 사용자 맞춤형 추천 헤더를 표시하는 기능이 추가되었습니다. 사용자 경험을 개인화하는 좋은 개선입니다.

404-420: 그라데이션 배경 및 추천 헤더 UI 추가

추천 영역에 그라데이션 배경과 persona 기반 추천 텍스트 헤더가 추가되었습니다. 시각적으로 섹션을 구분하고 개인화된 메시지를 제공하여 UX가 향상되었습니다.

data/place/src/main/java/com/teamsolply/solply/place/dto/response/GetPlacesResponseDto.kt (3)

23-24: 필드명 변경 확인

mainTag에서 primaryTag로 필드명이 변경되었습니다. 백엔드 API 스펙 변경에 따른 것으로 보이며, 요약에 따르면 관련 매퍼(PlaceEntityMapper, PlaceRepositoryImpl)도 함께 업데이트되었다고 명시되어 있습니다.

29-30: 새로운 필수 필드 추가

townId 필드가 추가되었습니다. 백엔드 API가 이 필드를 항상 제공하는지, 기존 직렬화된 데이터와의 호환성 문제가 없는지 확인해주세요.

21-21: 리뷰 의견이 부정확합니다

검증 결과, thumbnailImageUrl 필드 변경은 null 처리 문제를 야기하지 않습니다.

GetPlacesResponseDto.PlaceDto (String?) → PlaceEntity (String?): 타입 일치

PlaceViewModel에서 PlaceData.thumbnailUrl (String?): null 안전

RecommendPlaceInfo와 SearchResultEntity는 별도 DTO 사용으로 영향 없음

이 변경은 데이터 계층 내에서 일관성 있게 처리되며, UI 컴포넌트는 이미 nullable 값을 안전하게 처리합니다.

feature/search/src/main/java/com/teamsolply/solply/registerplace/RegisterPlaceScreen.kt (3)

221-221: 상단 패딩 증가

상단 패딩이 16.dp에서 50.dp로 증가했습니다. 시스템 상태 표시줄 영역을 고려한 조정으로 보이며, PR 전반에 걸쳐 일관되게 적용된 것으로 확인됩니다.

536-540: Shape 개선

키워드 버튼의 shape을 RoundedCornerShape(16.dp)에서 CircleShape으로 변경했습니다. 더 명확한 의미를 전달하며 시각적으로도 적절한 개선입니다.

129-131: 선택된 URI 리셋 기능 추가

resetSelectedUris 콜백이 추가되어 특정 인덱스의 이미지를 제거할 수 있게 되었습니다. Composition 계층을 통해 적절히 전달되고 있으며 구현이 깔끔합니다.

Also applies to: 175-175, 462-462

feature/maps/src/main/java/com/teamsolply/solply/maps/MapsScreen.kt (3)

484-487: 지도 UI 설정 추가

네이버 지도의 기본 줌 컨트롤과 나침반을 비활성화했습니다. 커스텀 UI를 위한 적절한 설정입니다.

286-286: 코스 편집 및 이름 변경 플로우 개선

finishEditCourseClick와 onChangeRenameCourse 콜백이 추가되어 편집 완료 및 코스 정보 업데이트 흐름이 개선되었습니다. Composition 계층을 통해 일관되게 전달되고 있습니다.

Also applies to: 355-357, 404-404, 658-658

335-337: 제보 이미지 리셋 기능 추가

resetSelectedUris 콜백이 ReportPlaceDialog에 추가되어 개별 이미지 제거가 가능해졌습니다. Intent 시스템과 잘 통합되어 있습니다.

feature/maps/src/main/java/com/teamsolply/solply/maps/MapsViewModel.kt (3)

269-275: 코스 이름 변경 UI 상태 초기화

바텀시트를 열 때 현재 코스 정보를 편집 필드에 미리 로드합니다. 사용자 경험을 개선하는 좋은 구현입니다.

277-284: 코스 정보 업데이트 로직 추가

ChangeRenameCourse Intent가 추가되어 코스명과 소개를 즉시 업데이트할 수 있습니다. 구현이 깔끔합니다.

337-342: URI 리셋 로직 구현

ResetSelectedUris Intent가 추가되어 특정 인덱스의 이미지를 제거할 수 있습니다. 경계 검사를 포함하여 안전하게 구현되어 있습니다.

feature/maps/src/main/java/com/teamsolply/solply/maps/component/bottomsheet/RenameCourseBottomSheet.kt (3)

128-132: 입력 길이 제한 구현

코스 이름(18자)과 소개(20자)에 대한 입력 길이 제한이 적절히 구현되어 있습니다.

Also applies to: 153-157

135-142: 글자 수 표시 추가

입력 필드 하단에 현재 글자 수를 표시하여 사용자에게 명확한 피드백을 제공합니다. 좋은 UX 개선입니다.

Also applies to: 160-166

98-118: 헤더 레이아웃 개선

헤더를 Box에서 Row로 변경하여 더 의미론적으로 명확한 레이아웃 구조를 제공합니다.

feature/maps/src/main/java/com/teamsolply/solply/maps/component/dialog/ReportPlaceDialog.kt (3)

90-103: Dialog에서 Popup으로 전환

Dialog 대신 Popup을 사용하여 더 세밀한 동작 제어가 가능해졌습니다. WindowInsets 처리를 통해 시스템 UI와의 겹침을 방지하고, dismissOnClickOutside = false 설정으로 실수로 닫히는 것을 방지합니다.

중첩된 Box 구조가 다소 복잡해 보이지만, Popup의 전체 화면 레이아웃과 적절한 패딩 처리를 위해 필요한 것으로 판단됩니다.

Also applies to: 104-240

81-81: 이미지 리셋 기능 통합

resetSelectedUris 콜백이 ReportPlaceDialog → ReportContentScreen → RegisterPlaceImage로 적절히 전달되어 제보 다이얼로그에서도 이미지 제거가 가능해졌습니다.

Also applies to: 191-191, 298-298, 366-366

145-170: 헤더 네비게이션 추가

뒤로가기 버튼과 제목이 포함된 헤더가 추가되었습니다. 페이저 페이지에 따라 동작이 적절히 변경됩니다 (첫 페이지: 닫기, 두 번째 페이지: 이전으로).

coderabbitai · 2025-11-20T12:57:49Z

data/course/src/main/java/com/teamsolply/solply/course/favoriteTown/dto/TownTreeResponseDto.kt

+    @SerialName("townName") val townName: String,
+    @SerialName("parentTownId") val parentTownId: Long?


⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

DTO 구조 변경 및 필수 필드 추가

TownNodeDto가 중첩 구조에서 평면 구조로 변경되었습니다:

townName이 nullable에서 non-nullable로 변경 (필수 필드)

parentTownId 추가 (부모-자식 관계 표현)

기존 name, subTowns 필드 제거

백엔드 API가 townName을 항상 제공하는지 확인해주세요. null이 전송되면 역직렬화 오류가 발생합니다.

🏁 Script executed:

#!/bin/bash # TownNodeDto 사용처 확인 rg -nP --type=kotlin -C3 'TownNodeDto' -g '!**/test/**'

Length of output: 1378

Let me check the complete TownNodeDto definition to see all fields:Based on my verification of the Kotlin serialization behavior and the code structure, I can now generate the rewritten review comment:

백엔드 API 응답 검증 필수

townName: String은 필수 필드입니다. kotlinx.serialization의 기본 설정(explicitNulls=true)에서 non-nullable 필드는 JSON에 반드시 포함되어야 하므로, 백엔드가 이 필드를 항상 제공해야 합니다.

또한 parentTownId: Long?은 현재 기본값이 없는 nullable 필드이므로, 기본값이 있어야 kotlinx.serialization에서 선택적 필드로 취급됩니다. 최상위 지역(부모가 없는)의 경우를 대비하여 parentTownId: Long? = null로 수정하는 것을 권장합니다.

백엔드 API 응답에서 townName이 항상 존재하고 null이 아닌지 확인

parentTownId가 선택적 필드인 경우 기본값 할당 권장

🤖 Prompt for AI Agents

In data/course/src/main/java/com/teamsolply/solply/course/favoriteTown/dto/TownTreeResponseDto.kt around lines 14-15, the DTO declares townName: String (non-nullable) and parentTownId: Long? (nullable without default); ensure the backend always supplies a non-null townName, and make parentTownId optional by giving it a default value so kotlinx.serialization treats it as optional — change parentTownId to have a default of null (parentTownId: Long? = null) and verify the API always returns a non-null townName.

coderabbitai · 2025-11-20T12:57:49Z

...ps/src/main/java/com/teamsolply/solply/maps/component/bottomsheet/RenameCourseBottomSheet.kt

+    val isButtonEnabled = newCourseName.isNotEmpty() &&
+        newCourseName.length != 18 &&
+        newCourseIntroduction.isNotEmpty() &&
+        newCourseIntroduction.length != 20


⚠️ Potential issue | 🔴 Critical

버튼 활성화 로직 오류

버튼 활성화 조건에 논리 오류가 있습니다:

Line 65: newCourseName.length != 18 - 길이가 정확히 18일 때만 버튼이 비활성화됩니다

Line 67: newCourseIntroduction.length != 20 - 길이가 정확히 20일 때만 버튼이 비활성화됩니다

이는 의도한 동작이 아닌 것으로 보입니다. 아마도 최대 길이 제한을 체크하려고 했다면 다음과 같이 수정해야 합니다:

- val isButtonEnabled = newCourseName.isNotEmpty() && - newCourseName.length != 18 && - newCourseIntroduction.isNotEmpty() && - newCourseIntroduction.length != 20 + val isButtonEnabled = newCourseName.isNotEmpty() && + newCourseName.length <= 18 && + newCourseIntroduction.isNotEmpty() && + newCourseIntroduction.length <= 20

🤖 Prompt for AI Agents

In feature/maps/src/main/java/com/teamsolply/solply/maps/component/bottomsheet/RenameCourseBottomSheet.kt around lines 64 to 67, the button-enabled condition incorrectly uses != to check lengths (disabling only when length equals exactly 18 or 20); change those checks to enforce maximum length (e.g., newCourseName.length <= 18 and newCourseIntroduction.length <= 20) while keeping the non-empty checks so the button is enabled only when both fields are non-empty and within their max lengths.

coderabbitai · 2025-11-20T12:57:49Z

feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/screen/AllowClauseScreen.kt

+            onClick = { onNextClick() },
            selected = isAllChecked,


⚠️ Potential issue | 🔴 Critical

필수 약관 동의를 우회할 수 있는 중요한 문제입니다.

"다음" 버튼의 onClick이 isAllChecked 상태와 관계없이 항상 실행되도록 변경되었습니다. 이는 사용자가 필수 약관(만 14세 이상, 서비스 이용 약관, 개인정보 처리방침)에 동의하지 않고도 다음 단계로 진행할 수 있음을 의미합니다. 이는 법적/규정 준수 문제를 초래할 수 있습니다.

다음과 같이 조건부 가드를 복원하세요:

SolplyBasicButton( text = "다음", modifier = Modifier.padding(bottom = 24.dp), - onClick = { onNextClick() }, + onClick = { if (isAllChecked) onNextClick() }, selected = isAllChecked, textStyle = SolplyTheme.typography.button16M, textColor = if (isAllChecked) SolplyTheme.colors.white else SolplyTheme.colors.gray800, enabledBackgroundColor = SolplyTheme.colors.gray900, disabledBackgroundColor = SolplyTheme.colors.gray300 )

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

onClick = { onNextClick() },

selected = isAllChecked,

onClick = { if (isAllChecked) onNextClick() },

selected = isAllChecked,

🤖 Prompt for AI Agents

In feature/onboarding/src/main/java/com/teamsolply/solply/onboarding/screen/AllowClauseScreen.kt around lines 140-141, the "Next" button's onClick handler is always invoking onNextClick regardless of isAllChecked; restore the conditional guard so the handler only proceeds when the user has checked all required consents (e.g., only call onNextClick when isAllChecked is true), and ensure the button is non-actionable otherwise (disable the button or no-op the click) to prevent bypassing mandatory agreements.

coderabbitai · 2025-11-20T12:57:50Z

feature/search/src/main/java/com/teamsolply/solply/search/SearchScreen.kt

+                    sideEffect.townId,
+                    sideEffect.placeId


⚠️ Potential issue | 🔴 Critical

네비게이션 파라미터 순서 오류 - 수정 필요

navigateToPlaceDetail 함수 호출 시 파라미터 순서가 변경되었습니다. 함수 시그니처는 (String, Long, Long)이며 (type, placeId, townId) 순서로 받는 것으로 보이는데, 현재 코드는 (type, townId, placeId) 순서로 전달하고 있습니다.

이는 잘못된 장소로 이동하거나 런타임 에러를 발생시킬 수 있는 치명적인 버그입니다.

다음과 같이 수정해야 합니다:

is SearchSideEffect.NavigateToPlaceDetail -> navigateToPlaceDetail( MapsType.PLACE_DETAIL.name, - sideEffect.townId, - sideEffect.placeId + sideEffect.placeId, + sideEffect.townId )

함수 시그니처를 확인하여 올바른 파라미터 순서를 적용해주세요.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

sideEffect.townId,

sideEffect.placeId

is SearchSideEffect.NavigateToPlaceDetail -> navigateToPlaceDetail(

MapsType.PLACE_DETAIL.name,

sideEffect.placeId,

sideEffect.townId

)

🤖 Prompt for AI Agents

In feature/search/src/main/java/com/teamsolply/solply/search/SearchScreen.kt around lines 64-65, the call passes navigation parameters in the wrong order (sideEffect.townId, sideEffect.placeId) while the navigateToPlaceDetail signature expects (type: String, placeId: Long, townId: Long); swap the second and third arguments so you call navigateToPlaceDetail(type, sideEffect.placeId, sideEffect.townId) and ensure the types match the function signature.

ImHyungsuk

굿굿

coderabbitai

Actionable comments posted: 0

🧹 Nitpick comments (1)

feature/place/src/main/java/com/teamsolply/solply/place/PlaceViewModel.kt (1)

165-180: 병렬 로딩으로 성능 개선이 좋습니다만, 에러 핸들링을 고려해보세요.

세 개의 데이터 로딩 작업을 병렬로 실행하여 초기 로드 시간을 개선한 것은 좋은 변경입니다. 다만 async 블록 중 하나라도 실패하면 await() 호출 시 예외가 전파되어 다른 작업들의 결과를 받지 못하고 상태가 불완전하게 남을 수 있습니다.

부분 실패 시에도 성공한 데이터는 표시할 수 있도록 개별 try-catch 처리를 고려해보시기 바랍니다.

🔎 에러 핸들링 개선 예시

-                    val placesJob = async {
-                        fetchPlaces(
-                            townId = userInfo.selectedTown.townId,
-                            mainTagId = null,
-                            subTagAIdList = null,
-                            subTagBIdList = null
-                        )
-                    }
-                    val recommendJob = async {
-                        fetchRecommendPlace(townId = userInfo.selectedTown.townId)
-                    }
-                    val mainTagsJob = async { fetchMainTags() }
-
-                    placesJob.await()
-                    recommendJob.await()
-                    mainTagsJob.await()
+                    val placesJob = async {
+                        runCatching {
+                            fetchPlaces(
+                                townId = userInfo.selectedTown.townId,
+                                mainTagId = null,
+                                subTagAIdList = null,
+                                subTagBIdList = null
+                            )
+                        }
+                    }
+                    val recommendJob = async {
+                        runCatching {
+                            fetchRecommendPlace(townId = userInfo.selectedTown.townId)
+                        }
+                    }
+                    val mainTagsJob = async {
+                        runCatching { fetchMainTags() }
+                    }
+
+                    placesJob.await()
+                    recommendJob.await()
+                    mainTagsJob.await()

📜 Review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ac7ced2 and a06261e.

📒 Files selected for processing (1)

feature/place/src/main/java/com/teamsolply/solply/place/PlaceViewModel.kt

🔇 Additional comments (2)

feature/place/src/main/java/com/teamsolply/solply/place/PlaceViewModel.kt (2)

12-12: LGTM!

병렬 실행을 위해 필요한 import가 적절하게 추가되었습니다.

256-256: PlaceType 매핑 로직이 올바르게 구현되었습니다.

valueOf에서 fromApiName으로의 변경은 적절한 수정입니다. fromApiName 메서드는 API 응답의 한글 이름을 PlaceType 열거형으로 매핑하며, 알 수 없는 API 이름을 받은 경우 ALL로 기본값을 설정하여 안전하게 처리합니다. valueOf처럼 예외를 발생시키지 않고 graceful하게 처리하는 구현입니다.

leeseokchan00 added 15 commits November 18, 2025 20:55

refactor#154 place detail api end point

f0ffbd8

refactor#154 inner padding

e3fb92d

refactor#154 place header color

5614da9

refactor#154 text style

373249e

refactor#154 report dialog to popup

42ec775

refactor#154 selected uri delete

7492212

refactor#154 report bottom padding

fefcfad

refactor#154 rename bottomsheet

63fc451

refactor#154 edit course bottomsheet title padding

6e8d89e

refactor#154 favorite town end point

4f89672

refactor#154 mypage top padding

9492eb6

refactor#154 collection top padding

0bedb32

refactor#154 button selected state

a3507f5

refactor#154 async api

a7b5102

chore#154 ktlint

ac7ced2

leeseokchan00 self-assigned this Nov 20, 2025

coderabbitai bot reviewed Nov 20, 2025

View reviewed changes

Merge branch 'develop' into feature/#154-first-qa-2

b85c1ee

ImHyungsuk approved these changes Jan 6, 2026

View reviewed changes

feature#154 PlaceType 메소드 오류 수정

a06261e

coderabbitai bot reviewed Jan 6, 2026

View reviewed changes

ImHyungsuk merged commit d7cdb5f into develop Jan 6, 2026
1 check passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Feature/#154] first qa-2 #158

[Feature/#154] first qa-2 #158

Uh oh!

leeseokchan00 commented Nov 20, 2025 •

edited by coderabbitai bot

Loading

Uh oh!

coderabbitai bot commented Nov 20, 2025 •

edited

Loading

Uh oh!

coderabbitai bot left a comment

Uh oh!

coderabbitai bot Nov 20, 2025

Uh oh!

coderabbitai bot Nov 20, 2025

Uh oh!

coderabbitai bot Nov 20, 2025

Uh oh!

coderabbitai bot Nov 20, 2025

Uh oh!

ImHyungsuk left a comment

Uh oh!

coderabbitai bot left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		@SerialName("townName") val townName: String,
		@SerialName("parentTownId") val parentTownId: Long?

[Feature/#154] first qa-2 #158

[Feature/#154] first qa-2 #158

Uh oh!

Conversation

leeseokchan00 commented Nov 20, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📌 PR 요약

🌱 PR 포인트

📸 스크린샷

📮 관련 이슈

Summary by CodeRabbit

릴리스 노트

Uh oh!

coderabbitai bot commented Nov 20, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Possibly related PRs

Suggested labels

Suggested reviewers

Poem

Pre-merge checks and finishing touches

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot Nov 20, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot Nov 20, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot Nov 20, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot Nov 20, 2025

Choose a reason for hiding this comment

Uh oh!

ImHyungsuk left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

leeseokchan00 commented Nov 20, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Nov 20, 2025 •

edited

Loading