🔗 :: (#462) 관심 분야 선택 기능 구현 #465

uson1004 · 2025-05-20T23:41:30Z

개요

작업 내용

관심 분야 기능을 디자인에 맞게 퍼블리싱 하였습니다
관심있는 전공을 입력하여 모집의뢰서가 추가되면 알람을 보내주도록 설계되었습니다

할 말

서비스를 더욱 깊게 이해하게 되었습니다

Summary by CodeRabbit

신규 기능
- 관심 분야 설정 완료 화면이 추가되어, 관심 분야 선택 완료 시 맞춤형 완료 메시지와 성공 아이콘이 표시됩니다.
- 관심 분야 선택 화면이 리뉴얼되어, 여러 전공을 선택할 수 있고, 선택한 전공에 따라 버튼 상태와 안내 메시지가 동적으로 변경됩니다.
- 관심 분야와 관련된 채용 공고 정보를 확인할 수 있습니다.
- 관심 분야 관련 API, 데이터 모델, 저장소, 유즈케이스 및 네트워크 연동이 새롭게 도입되어 안정적인 데이터 처리와 비동기 작업이 가능해졌습니다.
- 관심 분야 선택 화면에서 선택 완료 시 해당 학생 이름을 전달하여 완료 화면으로 이동하는 내비게이션 기능이 추가되었습니다.
개선 사항
- 마이페이지에서 관심 분야 항목 클릭 시 관심 분야 설정 화면으로 이동할 수 있습니다.
- 앱 내 내비게이션이 개선되어 관심 분야 완료 및 마이페이지 이동 시 상태가 보존되고 중복 화면 생성이 방지됩니다.
디자인 및 리소스
- 성공 상태를 나타내는 새로운 아이콘이 추가되었습니다.
- 관심 분야 관련 안내 문구 및 버튼 텍스트가 개선되었습니다.
버그 수정
- 관심 분야 등록 실패 시 안내 메시지가 표시됩니다.

… feature/462-관심-분야-선택-기능-구현

coderabbitai · 2025-05-20T23:41:38Z

"""

Walkthrough

이 변경 사항은 관심사(Interests) 기능의 전체적인 도메인, 데이터, 네트워크, UI 계층을 신규로 도입하고, 관심사 설정 완료 화면 및 내비게이션을 추가합니다. 관심사 관련 엔티티, 리포지토리, 데이터소스, API, UI 컴포저블, 뷰모델, 문자열 및 디자인 리소스가 포함되어 있습니다.

Changes

파일/경로 그룹	변경 요약
`app/src/main/java/team/retum/jobisandroidv2/JobisNavigator.kt`, `.../navigation/MainNavigation.kt`	관심사 완료 내비게이션 함수 및 목적지 추가
`core/common/src/main/java/team/retum/common/utils/ResourceKeys.kt`	`STUDENT_NAME` 상수 추가
`core/data/src/main/java/team/retum/data/di/RepositoryModule.kt`	`InterestsRepository` DI 바인딩 추가
`core/data/src/main/java/team/retum/data/repository/interests/InterestsRepository.kt`, `InterestsRepositoryImpl.kt`	관심사 리포지토리 인터페이스 및 구현체 추가
`core/design-system/src/main/res/drawable/success.xml`	성공 상태 아이콘 벡터 리소스 추가
`core/domain/src/main/java/team/retum/usecase/entity/interests/InterestsEntity.kt`, `InterestsRecruitmentEntity.kt`	관심사 및 관심사 채용 엔티티, 변환 함수 추가
`core/domain/src/main/java/team/retum/usecase/usecase/interests/FetchInterestsUseCase.kt`, `FetchInterestsSearchRecruitmentsUseCase.kt`, `SetInterestsToggleUseCase.kt`	관심사 관련 유스케이스 3종 추가
`core/network/src/main/java/team/retum/network/api/InterestsApi.kt`, `datasource/interests/InterestsDataSource.kt`, `InterestsDataSourceImpl.kt`	관심사 API, 데이터소스 인터페이스 및 구현체 추가
`core/network/src/main/java/team/retum/network/di/NetworkModule.kt`, `RemoteDataSourceModule.kt`	관심사 API 및 데이터소스 DI 바인딩 추가
`core/network/src/main/java/team/retum/network/di/RequestUrls.kt`	관심사 관련 URL 상수 추가
`core/network/src/main/java/team/retum/network/model/request/interests/InterestsToggleRequest.kt`	관심사 토글 요청 모델 추가
`core/network/src/main/java/team/retum/network/model/response/interests/FetchInterestsResponse.kt`, `FetchInterestsRecruitmentsResponse.kt`	관심사 및 채용 응답 모델 추가
`feature/interests/build.gradle.kts`	`core:network` 의존성 추가
`feature/interests/src/main/java/team/retum/jobis/interests/navigation/InterestsCompleteNavigation.kt`, `InterestsNavigation.kt`	관심사 완료 내비게이션 및 목적지, 콜백 추가
`feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsCompleteScreen.kt`	관심사 설정 완료 UI 화면 추가
`feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsScreen.kt`	관심사 선택 화면 리팩토링, 상태 기반 UI, 애니메이션 추가
`feature/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsViewmodel.kt`	관심사 뷰모델, 상태, 사이드이펙트 클래스 추가
`feature/interests/src/main/res/values/strings.xml`	관심사 관련 문자열 리소스 추가 및 수정
`feature/mypage/src/main/java/team/retum/jobis/navigation/MyPageNavigation.kt`, `ui/MyPageScreen.kt`	마이페이지 이동 내비게이션 개선, 관심사 클릭 시 콜백 연결
`feature/landing/src/main/java/team/retum/landing/LandingViewModel.kt`	신규 빈 ViewModel 클래스 추가

Sequence Diagram(s)

sequenceDiagram
    participant User
    participant MyPageScreen
    participant Navigator
    participant InterestsScreen
    participant InterestsViewmodel
    participant Repository
    participant DataSource
    participant API

    User->>MyPageScreen: "관심 분야" 클릭
    MyPageScreen->>Navigator: onSelectInterestClick()
    Navigator->>InterestsScreen: navigateToInterests()
    InterestsScreen->>InterestsViewmodel: 초기화
    InterestsViewmodel->>Repository: fetchInterests()
    Repository->>DataSource: fetchInterests()
    DataSource->>API: GET /interests
    API-->>DataSource: 관심사 데이터 반환
    DataSource-->>Repository: 데이터 반환
    Repository-->>InterestsViewmodel: 데이터 반환
    InterestsViewmodel-->>InterestsScreen: 상태 업데이트
    User->>InterestsScreen: 관심사 선택 & 완료 버튼 클릭
    InterestsScreen->>InterestsViewmodel: patchInterestsMajor()
    InterestsViewmodel->>Repository: setInterestsToggle()
    Repository->>DataSource: setInterestsToggle()
    DataSource->>API: PATCH /interests
    API-->>DataSource: 성공/실패 응답
    DataSource-->>Repository: 응답 반환
    Repository-->>InterestsViewmodel: 응답 반환
    InterestsViewmodel-->>InterestsScreen: 성공 시 MoveToInterestsComplete SideEffect
    InterestsScreen->>Navigator: navigateToInterestsComplete(studentName)

Suggested reviewers

parkuiery
Tmdhoon2

Poem

🐰
관심 분야 토글, 새로운 길
학생 이름 불러주며 화면이 빛나네
성공 아이콘 반짝, 알림도 챙겨
내비게이션 따라 hop! hop!
관심사 등록, 토끼도 기뻐
코드 한 줄 한 줄 봄내음 가득
앞으로도 hop-hop, 성장하길!
🥕
"""

📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 68147b9 and ab502e2.

📒 Files selected for processing (1)

feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsCompleteScreen.kt (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsCompleteScreen.kt

⏰ Context from checks skipped due to timeout of 90000ms (2)

GitHub Check: lint_and_build (build)
GitHub Check: lint_and_build (lint)

✨ 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

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

Review comments: Directly reply to a review comment made by CodeRabbit. Example:
- I pushed a fix in commit <commit_id>, please review it.
- Explain this complex logic.
- Open a follow-up GitHub issue for this discussion.
Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
- @coderabbitai explain this code block.
- @coderabbitai modularize this function.
PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
- @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
- @coderabbitai read src/utils.ts and explain its main purpose.
- @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
- @coderabbitai help me debug CodeRabbit configuration file.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

@coderabbitai pause to pause the reviews on a PR.
@coderabbitai resume to resume the paused reviews.
@coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
@coderabbitai full review to do a full review from scratch and review all the files again.
@coderabbitai summary to regenerate the summary of the PR.
@coderabbitai generate docstrings to generate docstrings for this PR.
@coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
@coderabbitai resolve resolve all the CodeRabbit review comments.
@coderabbitai configuration to show the current CodeRabbit configuration for the repository.
@coderabbitai help to get help.

Other keywords and placeholders

Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (`.coderabbit.yaml`)

You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
Please see the configuration documentation for more information.
If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

Visit our Documentation for detailed information on how to use CodeRabbit.
Join our Discord Community to get help, request features, and share feedback.
Follow us on X/Twitter for updates and announcements.

coderabbitai

Actionable comments posted: 5

🧹 Nitpick comments (13)

core/network/src/main/java/team/retum/network/datasource/interests/InterestsDataSource.kt (1)
8-8: 파라미터 이름을 좀 더 명확하게 하는 것이 좋겠습니다.

codes 라는 파라미터 이름보다는 request 또는 toggleRequest와 같이 좀 더 명확한 이름을 사용하면 코드의 가독성이 향상될 것입니다.
-    suspend fun setInterestsToggle(codes: InterestsToggleRequest)
+    suspend fun setInterestsToggle(request: InterestsToggleRequest)
core/data/src/main/java/team/retum/data/repository/interests/InterestsRepository.kt (2)
9-10: Repository가 네트워크 모델을 직접 반환합니다.

일반적으로 Repository는 네트워크 모델을 도메인 엔티티로 변환하여 반환하는 것이 좋습니다. 현재 구현에서는 UseCase에서 변환 작업을 수행하고 있지만, 이 책임을 Repository로 이동하는 것을 고려해 보세요.
-    suspend fun fetchInterests(): FetchInterestsResponse
-    suspend fun fetchInterestsSearchRecruitments(): FetchInterestsRecruitmentsResponse
+    suspend fun fetchInterests(): InterestsEntity
+    suspend fun fetchInterestsSearchRecruitments(): InterestsRecruitmentsEntity
8-8: 파라미터 이름을 좀 더 명확하게 하는 것이 좋겠습니다.

codes 대신 request 또는 toggleRequest와 같은 이름을 사용하면 코드 가독성이 향상될 것입니다.
-    suspend fun setInterestsToggle(codes: InterestsToggleRequest)
+    suspend fun setInterestsToggle(request: InterestsToggleRequest)
core/network/src/main/java/team/retum/network/api/InterestsApi.kt (1)
13-15: 파라미터 이름을 좀 더 명확하게 하는 것이 좋겠습니다.

codes 대신 request 또는 toggleRequest와 같은 좀 더 명확한 이름을 사용하면 코드 가독성이 향상될 것입니다.
-    suspend fun setInterestsToggle(
-        @Body codes: InterestsToggleRequest,
-    )
+    suspend fun setInterestsToggle(
+        @Body request: InterestsToggleRequest,
+    )
feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsCompleteScreen.kt (3)
46-49: 문자열 리소스 사용 권장

TopAppBar의 타이틀에 하드코딩된 문자열 "관심 분야 설정"을 사용하고 있습니다. 언어 지원과 일관성을 위해 strings.xml의 리소스(R.string.set_interests)를 사용하는 것이 더 적합할 것 같습니다.
-            title = "관심 분야 설정",
+            title = stringResource(R.string.set_interests),
51-54: 더 명확한 contentDescription 권장

이미지의 contentDescription이 "major_check_success"로 되어 있는데, 이는 접근성 측면에서 이미지의 목적을 명확히 설명하지 않습니다. 사용자가 관심 분야 선택을 완료했음을 나타내는 더 설명적인 텍스트를 사용하는 것이 좋습니다.
-            contentDescription = "major_check_success",
+            contentDescription = "관심 분야 선택 완료 성공 아이콘",
50-76: 하드코딩된 패딩 값

UI 요소에 하드코딩된 dp 값들(188.dp, 20.dp 등)이 사용되고 있습니다. 일관된 디자인 시스템을 위해 이러한 값들을 dimens.xml 파일이나 디자인 시스템의 정의된 상수로 추출하는 것을 고려해 보세요.
core/network/src/main/java/team/retum/network/datasource/interests/InterestsDataSourceImpl.kt (2)
19-23: 반환 스타일 간소화 검토

코드가 기능적으로는 완벽하지만, 다음과 같이 간소화할 수도 있습니다:
-    override suspend fun fetchInterests(): FetchInterestsResponse {
-        return RequestHandler<FetchInterestsResponse>().request {
-            interestsApi.fetchInterests()
-        }
-    }
+    override suspend fun fetchInterests(): FetchInterestsResponse =
+        RequestHandler<FetchInterestsResponse>().request {
+            interestsApi.fetchInterests()
+        }
25-29: 반환 스타일 간소화 검토

앞선 코멘트와 동일하게, 여기도 간소화할 수 있습니다:
-    override suspend fun fetchInterestsSearchRecruitments(): FetchInterestsRecruitmentsResponse {
-        return RequestHandler<FetchInterestsRecruitmentsResponse>().request {
-            interestsApi.fetchInterestsSearchRecruitments()
-        }
-    }
+    override suspend fun fetchInterestsSearchRecruitments(): FetchInterestsRecruitmentsResponse =
+        RequestHandler<FetchInterestsRecruitmentsResponse>().request {
+            interestsApi.fetchInterestsSearchRecruitments()
+        }
core/domain/src/main/java/team/retum/usecase/entity/interests/InterestsEntity.kt (1)
11-16: 중첩 엔티티에도 @Immutable 어노테이션을 부여해 주세요

InterestsEntity 본체에는 @Immutable이 달려 있지만, 내부 InterestMajorEntity에는 달려 있지 않습니다.
Compose 상태가 이 객체까지 관찰하게 될 경우, 중첩 객체의 변경을 컴파일러가 추적하지 못해 불필요한 재구성을 유발할 수 있습니다. 동일한 불변성 계약을 명확히 하기 위해 다음과 같이 추가하는 편이 안전합니다.
 data class InterestsEntity(
     ...
 ) {
-    data class InterestMajorEntity(
+    @Immutable
+    data class InterestMajorEntity(
         val id: Int,
feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsScreen.kt (1)

60-69: 버튼 상태 계산 로직을 UI → ViewModel 로 이동하는 TODO 처리 권장

selectedMajorCount, buttonEnable, buttonText 계산은 순수 상태 파생값입니다.
현재 TODO 로 남겨두신 것처럼 ViewModel 또는 InterestsState 내에서 val 로 제공하면 UI 단계의 책임이 줄어들고, 테스트도 쉬워집니다.
feature/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsViewmodel.kt (2)
3-3: 사용되지 않는 import 제거

android.util.Log 는 파일 내에서 쓰이지 않아 빌드 경고가 발생합니다. 삭제해 주세요.

84-99: selectedMajorId 유지 방식이 해제 후에도 값을 남깁니다

토글 해제 시에도 selectedMajorId = majorId 로 덮어쓰면, UI 단에서 “마지막 클릭 전공”이 항상 설정되어 있습니다.
이 값이 다른 로직에 활용될 계획이 없다면 삭제하거나, 해제 시 0 으로 초기화하는 편이 명확합니다.
-    setState {
-        state.value.copy(
-            selectedMajorId = majorId,
-            selectedMajorCodes = currentSelected,
-        )
-    }
+    setState {
+        state.value.copy(
+            selectedMajorId = if (currentSelected.contains(majorId)) majorId else 0,
+            selectedMajorCodes = currentSelected,
+        )
+    }

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4dd3854 and 2e4b78d.

📒 Files selected for processing (32)

app/src/main/java/team/retum/jobisandroidv2/JobisNavigator.kt (2 hunks)
app/src/main/java/team/retum/jobisandroidv2/navigation/MainNavigation.kt (2 hunks)
core/common/src/main/java/team/retum/common/utils/ResourceKeys.kt (1 hunks)
core/data/src/main/java/team/retum/data/di/RepositoryModule.kt (2 hunks)
core/data/src/main/java/team/retum/data/repository/interests/InterestsRepository.kt (1 hunks)
core/data/src/main/java/team/retum/data/repository/interests/InterestsRepositoryImpl.kt (1 hunks)
core/design-system/src/main/res/drawable/success.xml (1 hunks)
core/domain/src/main/java/team/retum/usecase/entity/interests/InterestsEntity.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/entity/interests/InterestsRecruitmentEntity.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/usecase/interests/FetchInterestsSearchRecruitmentsUseCase.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/usecase/interests/FetchInterestsUseCase.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/usecase/interests/SetInterestsToggleUseCase.kt (1 hunks)
core/network/build.gradle.kts (1 hunks)
core/network/src/main/java/team/retum/network/api/InterestsApi.kt (1 hunks)
core/network/src/main/java/team/retum/network/datasource/interests/InterestsDataSource.kt (1 hunks)
core/network/src/main/java/team/retum/network/datasource/interests/InterestsDataSourceImpl.kt (1 hunks)
core/network/src/main/java/team/retum/network/di/NetworkModule.kt (2 hunks)
core/network/src/main/java/team/retum/network/di/RemoteDataSourceModule.kt (2 hunks)
core/network/src/main/java/team/retum/network/di/RequestUrls.kt (1 hunks)
core/network/src/main/java/team/retum/network/model/request/interests/InterestsToggleRequest.kt (1 hunks)
core/network/src/main/java/team/retum/network/model/response/interests/FetchInterestsRecruitmentsResponse.kt (1 hunks)
core/network/src/main/java/team/retum/network/model/response/interests/FetchInterestsResponse.kt (1 hunks)
feature/interests/build.gradle.kts (1 hunks)
feature/interests/src/main/java/team/retum/jobis/interests/navigation/InterestsCompleteNavigation.kt (1 hunks)
feature/interests/src/main/java/team/retum/jobis/interests/navigation/InterestsNavigation.kt (1 hunks)
feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsCompleteScreen.kt (1 hunks)
feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsScreen.kt (2 hunks)
feature/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsCompleteViewModel.kt (1 hunks)
feature/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsViewmodel.kt (1 hunks)
feature/interests/src/main/res/values/strings.xml (1 hunks)
feature/mypage/src/main/java/team/retum/jobis/navigation/MyPageNavigation.kt (1 hunks)
feature/mypage/src/main/java/team/retum/jobis/ui/MyPageScreen.kt (1 hunks)

🧰 Additional context used

🧬 Code Graph Analysis (5)

feature/interests/build.gradle.kts (1)

buildSrc/src/main/kotlin/GradlePluginExtensions.kt (1)

implementation (15-17)

app/src/main/java/team/retum/jobisandroidv2/navigation/MainNavigation.kt (2)

feature/interests/src/main/java/team/retum/jobis/interests/navigation/InterestsNavigation.kt (1)

interests (9-19)

feature/interests/src/main/java/team/retum/jobis/interests/navigation/InterestsCompleteNavigation.kt (1)

interestsComplete (12-28)

feature/interests/src/main/java/team/retum/jobis/interests/navigation/InterestsNavigation.kt (1)

feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsScreen.kt (1)

Interests (35-50)

feature/interests/src/main/java/team/retum/jobis/interests/navigation/InterestsCompleteNavigation.kt (1)

feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsCompleteScreen.kt (1)

InterestsComplete (26-35)

feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsCompleteScreen.kt (1)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/appbar/JobisTopAppBar.kt (1)

JobisSmallTopAppBar (124-144)

⏰ Context from checks skipped due to timeout of 90000ms (2)

GitHub Check: lint_and_build (build)
GitHub Check: lint_and_build (lint)

🔇 Additional comments (37)

feature/interests/build.gradle.kts (1)

24-24: 네트워크 모듈 의존성 추가가 적절합니다.

관심 분야 기능 구현을 위해 core:network 모듈에 대한 의존성을 추가한 것이 적절합니다. 이는 관심 분야 관련 API 호출 기능을 활용하기 위해 필요한 설정입니다.

core/common/src/main/java/team/retum/common/utils/ResourceKeys.kt (1)

25-25: 네비게이션 키 추가가 적절합니다.

STUDENT_NAME 상수를 추가하여 관심 분야 설정 완료 화면으로 학생 이름을 전달하기 위한 네비게이션 인자로 사용하는 것이 좋습니다. 상수로 관리하면 오타 방지와 유지보수에 도움이 됩니다.

core/network/build.gradle.kts (1)

35-35: 개발 서버 URL 업데이트가 적절합니다.

개발 서버 URL을 명시적으로 설정하여 관심 분야 관련 API를 올바르게 호출할 수 있도록 한 것이 좋습니다. 이렇게 하면 로컬 속성 파일에서 값을 지정하지 않아도 기본값으로 개발 서버를 사용할 수 있습니다.

core/design-system/src/main/res/drawable/success.xml (1)

1-9: 성공 상태를 나타내는 아이콘 리소스 추가가 적절합니다.

관심 분야 설정 완료 화면에서 사용할 체크마크 아이콘을 벡터 드로어블로 추가한 것이 좋습니다. 체크마크와 원형의 조합은 작업 완료를 시각적으로 잘 나타냅니다.

다만, 아이콘의 색상(#2F53FF)이 앱의 디자인 시스템과 일치하는지 확인하는 것이 좋습니다.

디자인 시스템에서 정의된 기본 색상 값과 일치하는지 확인해보세요.

core/network/src/main/java/team/retum/network/di/NetworkModule.kt (2)

25-25: InterestsApi 임포트 추가가 적절합니다.

관심 분야 기능 구현을 위해 새로 생성된 InterestsApi 인터페이스를 임포트하는 것이 적절합니다.

193-197: InterestsApi 제공 메서드 추가가 적절합니다.

Retrofit을 통해 InterestsApi 인스턴스를 생성하고 제공하는 메서드를 추가한 것이 적절합니다. 이는 Hilt 의존성 주입 시스템을 통해 관심 분야 관련 API를 앱 전체에서 사용할 수 있게 합니다.

기존 API 제공 메서드들과 동일한 패턴을 따르고 있어 일관성이 유지되었습니다.

app/src/main/java/team/retum/jobisandroidv2/JobisNavigator.kt (2)

19-20: 새로운 관심 분야 완료 화면 네비게이션 임포트 추가됨

관심 분야 완료 화면으로 이동하기 위한 네비게이션 기능 임포트가 적절히 추가되었습니다.

109-111: 새로운 관심 분야 완료 화면 네비게이션 함수 구현

navigateToInterestsComplete 함수가 적절히 구현되었습니다. 학생 이름을 매개변수로 받아 완료 화면에 전달하는 방식이 기존 네비게이션 패턴을 잘 따르고 있습니다.

feature/mypage/src/main/java/team/retum/jobis/ui/MyPageScreen.kt (1)

238-238: 관심 분야 기능 활성화

기존에 "업데이트 예정" 토스트 메시지를 표시하던 것에서 실제 관심 분야 선택 화면으로 이동하는 기능으로 변경되었습니다. 이제 사용자가 관심 분야를 선택할 수 있게 되었습니다.

core/network/src/main/java/team/retum/network/di/RequestUrls.kt (1)

137-142: 관심 분야 API 엔드포인트 추가

새로운 관심 분야 기능을 위한 API 엔드포인트가 적절히 추가되었습니다. 기존의 URL 구조 패턴을 잘 따르고 있으며, 관심 분야 API(interests)와 관심 분야 관련 채용 정보 API(interestsRecruitments)에 대한 엔드포인트가 정의되었습니다.

core/data/src/main/java/team/retum/data/di/RepositoryModule.kt (2)

25-26: 관심 분야 리포지토리 인터페이스 및 구현체 임포트

관심 분야 기능을 위한 리포지토리 인터페이스와 구현체가 적절히 임포트되었습니다.

110-112: 관심 분야 리포지토리 의존성 주입 설정

Dagger Hilt를 사용하여 InterestsRepository 인터페이스와 InterestsRepositoryImpl 구현체를 바인딩하는 코드가 적절히 추가되었습니다. 이를 통해 앱 전체에서 InterestsRepository를 주입받아 사용할 수 있게 되었습니다.

core/network/src/main/java/team/retum/network/di/RemoteDataSourceModule.kt (1)

25-26: 새로운 관심 분야 기능에 대한 데이터 소스 추가가 잘 수행되었습니다.

InterestsDataSource 인터페이스와 그 구현체인 InterestsDataSourceImpl에 대한 바인딩이 다른 데이터 소스들과 동일한 패턴으로 적절하게 추가되었습니다. 의존성 주입 설정이 일관성 있게 구현되어 있습니다.

Also applies to: 110-112

feature/interests/src/main/java/team/retum/jobis/interests/navigation/InterestsNavigation.kt (1)

9-12: 네비게이션 콜백 파라미터 추가가 적절히 이루어졌습니다.

관심 분야 완료 화면으로 이동하기 위한 navigateToInterestsComplete 콜백 파라미터가 잘 추가되었습니다. Interests 컴포저블에 해당 콜백을 전달하여 학생 이름을 기반으로 완료 화면으로 네비게이션하는 흐름이 명확하게 구현되었습니다.

Also applies to: 14-17

core/network/src/main/java/team/retum/network/model/request/interests/InterestsToggleRequest.kt (1)

6-9: 관심 분야 토글을 위한 요청 모델이 적절히 구현되었습니다.

InterestsToggleRequest 데이터 클래스는 사용자 관심 분야 코드 ID 목록을 JSON으로 직렬화하기 위한 Moshi 어노테이션이 올바르게 적용되어 있습니다. JSON 필드명 매핑을 위한 @Json(name = "code_ids") 어노테이션도 적절하게 사용되었습니다.

core/domain/src/main/java/team/retum/usecase/usecase/interests/FetchInterestsSearchRecruitmentsUseCase.kt (1)

7-12: 관심 분야 기반 채용 정보 조회 유스케이스가 클린 아키텍처 원칙에 맞게 구현되었습니다.

이 유스케이스는 다음과 같은 장점을 가지고 있습니다:

의존성 주입을 통해 InterestsRepository에 대한 결합도를 낮췄습니다.

runCatching을 사용하여 예외 처리를 적절히 구현했습니다.

toEntity() 확장 함수를 통해 네트워크 응답 모델을 도메인 엔티티로 변환하는 책임을 명확히 분리했습니다.

이러한 구현은 앱의 다른 유스케이스들과 일관성을 유지하며, 단일 책임 원칙을 잘 따르고 있습니다.

feature/mypage/src/main/java/team/retum/jobis/navigation/MyPageNavigation.kt (1)

33-39: 마이페이지 네비게이션 개선 확인

네비게이션 옵션을 추가하여 상태 관리를 개선한 것은 좋은 접근 방식입니다. saveState, launchSingleTop, restoreState 옵션을 통해 사용자가 마이페이지로 이동할 때 상태를 보존하고 중복 인스턴스 생성을 방지할 수 있습니다.

core/domain/src/main/java/team/retum/usecase/usecase/interests/SetInterestsToggleUseCase.kt (1)

1-13: 관심 분야 토글 유스케이스 구현 확인

클린 아키텍처 원칙을 잘 따르고 있습니다. 단일 책임을 가진 유스케이스로 적절히 구현되었으며, 예외 처리를 위한 runCatching 블록 사용은 좋은 방식입니다.

app/src/main/java/team/retum/jobisandroidv2/navigation/MainNavigation.kt (2)

13-13: 관심 분야 완료 화면 임포트 추가 확인

관심 분야 완료 화면의 네비게이션을 위한 임포트가 추가되었습니다.

79-85: 관심 분야 관련 네비게이션 추가 확인

관심 분야 설정 화면과 완료 화면에 대한 네비게이션이 적절하게 추가되었습니다. 코드가 잘 구조화되어 있고 필요한 콜백 함수를 올바르게 전달하고 있습니다.

core/network/src/main/java/team/retum/network/model/response/interests/FetchInterestsResponse.kt (1)

1-17: 관심 분야 응답 모델 구현 확인

API 응답을 위한 데이터 클래스가 적절하게 구현되었습니다. Moshi 어노테이션을 통한 JSON 직렬화/역직렬화 설정이 올바르게 되어 있으며, 필요한 모든 필드가 포함되어 있습니다.

core/network/src/main/java/team/retum/network/datasource/interests/InterestsDataSource.kt (1)

1-11: 잘 설계된 인터페이스입니다.

관심 분야 관련 데이터 소스 인터페이스가 적절히 구현되었습니다. suspend 함수를 사용하여 비동기 작업을 처리하고 있으며, 명확한 기능 분리를 하였습니다.

core/domain/src/main/java/team/retum/usecase/usecase/interests/FetchInterestsUseCase.kt (2)

1-13: 깔끔하게 구현된 UseCase입니다.

UseCase 패턴을 잘 활용하였으며, 단일 책임 원칙(SRP)을 준수하고 있습니다.
의존성 주입을 통해 Repository를 주입받고, runCatching을 사용하여 예외 처리를 적절히 수행하고 있습니다.

10-12: extension 함수를 활용한 좋은 구현입니다.

toEntity() 확장 함수를 사용하여 네트워크 응답을 도메인 엔티티로 변환하는 로직을 분리함으로써 코드의 가독성과 재사용성을 높였습니다.

core/data/src/main/java/team/retum/data/repository/interests/InterestsRepository.kt (1)

1-11: Repository 인터페이스가 잘 정의되었습니다.

Repository 패턴 구현이 적절히 이루어졌습니다. 관심 분야와 관련된 데이터 조작 작업이 명확하게 정의되어 있습니다.

core/network/src/main/java/team/retum/network/api/InterestsApi.kt (2)

11-22: API 인터페이스 구현이 잘 되었습니다.

Retrofit을 사용한 API 정의가 적절히 이루어졌습니다. 애노테이션을 통한 HTTP 메서드 및 엔드포인트 설정이 명확하게 되어 있습니다.

20-21: 함수 이름과 엔드포인트 명명이 일치하는지 확인해주세요.

함수 이름은 fetchInterestsSearchRecruitments이고 엔드포인트는 interestsRecruitments입니다. 이름이 약간 다른데, 의도적인 것인지 확인해주세요. 가능하다면 일관성 있게 통일하는 것이 좋습니다.

feature/interests/src/main/res/values/strings.xml (1)

3-8: 적절하게 정의된 문자열 리소스

문자열 리소스가 관심 분야 선택 기능에 맞게 잘 정의되었습니다. 사용자 이름을 동적으로 표시하기 위한 %s 플레이스홀더와 선택된 항목 개수를 보여주기 위한 포맷 문자열도 적절히 사용되었습니다.

core/domain/src/main/java/team/retum/usecase/entity/interests/InterestsRecruitmentEntity.kt (2)

6-19: 잘 구조화된 엔티티 클래스

엔티티 클래스가 관심 분야와 관련된 채용 정보를 표현하기 위해 적절히 설계되었습니다. @Immutable 어노테이션을 사용하여 UI에서 사용될 데이터의 불변성을 보장한 것은 좋은 방식입니다.

21-34: 확장 함수를 통한 깔끔한 변환 로직

네트워크 응답 모델을 도메인 엔티티로 변환하는 확장 함수가 깔끔하게 구현되었습니다. 이러한 접근 방식은 관심사 분리 원칙을 따르며 코드 가독성을 높여줍니다.

feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsCompleteScreen.kt (1)

26-35: 내부 구현을 은닉하는 좋은 Composable 패턴

외부에서 접근 가능한 Composable 함수가 내부 구현을 적절히 캡슐화하고 있습니다. 필요한 매개변수만 노출하여 깔끔한 API를 제공합니다.

feature/interests/src/main/java/team/retum/jobis/interests/navigation/InterestsCompleteNavigation.kt (2)

11-28: 적절한 네비게이션 구성

NavGraphBuilder 확장 함수를 통해 네비게이션 그래프에 관심 분야 완료 화면을 추가하는 접근 방식이 잘 구현되었습니다. 네비게이션 인자를 적절히 설정하고 추출하는 로직도 잘 작성되었습니다.

30-32: 편리한 네비게이션 확장 함수

NavController에 확장 함수를 추가하여 관심 분야 완료 화면으로 쉽게 이동할 수 있도록 한 접근 방식이 좋습니다. 코드 사용이 간결해지고 가독성이 향상됩니다.

core/data/src/main/java/team/retum/data/repository/interests/InterestsRepositoryImpl.kt (1)

1-21: 코드가 잘 구현되었습니다.

Repository 구현이 깔끔하고 적절하게 설계되었습니다. 의존성 주입을 위한 @Inject 어노테이션이 올바르게 적용되었으며, 각 메서드는 DataSource에 작업을 명확하게 위임하고 있습니다. suspend 함수를 사용하여 비동기 작업도 적절히 처리하고 있습니다.

core/network/src/main/java/team/retum/network/datasource/interests/InterestsDataSourceImpl.kt (1)

13-17: 네트워크 요청 처리가 잘 구현되었습니다.

RequestHandler를 사용한 에러 처리 패턴이 일관되게 적용되었습니다. Unit 타입의 반환값을 가진 요청도 올바르게 처리하고 있습니다.

feature/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsCompleteViewModel.kt (1)

22-22: 빈 SideEffect 클래스 확인 필요

InterestsCompleteSideEffect 클래스가 비어 있습니다. 이 화면에서 사이드 이펙트가 필요하지 않다면 문제 없지만, 향후 확장을 위해 준비한 것인지 확인이 필요합니다.
core/network/src/main/java/team/retum/network/model/response/interests/FetchInterestsRecruitmentsResponse.kt (1)

10-18: 응답 모델에 회사 이름 필드 추가 검토

현재 InterestRecruitments 클래스에는 회사 ID(companyId)는 있지만 회사 이름이 없습니다. UI에서 회사 이름을 표시해야 할 경우, 별도의 API 호출이 필요하거나 다른 방식으로 회사 이름을 가져와야 할 수 있습니다. 필요하다면 응답 모델에 회사 이름 필드를 추가하는 것이 좋을 수 있습니다.

회사 이름이 UI에 필요한지, 그리고 현재 구현에서 어떻게 회사 이름을 표시하고 있는지 확인해 보세요. 회사 이름이 필요하다면 다음과 같이 필드를 추가할 수 있습니다:
data class InterestRecruitments(
    @Json(name = "id") val id: Int,
    @Json(name = "company_id") val companyId: String,
+   @Json(name = "company_name") val companyName: String,
    @Json(name = "company_profile_url") val companyProfileUrl: String,
    @Json(name = "train_pay") val trainPay: Int,
    @Json(name = "military_support") val militarySupport: Boolean,
    @Json(name = "hiring_jobs") val hiringJobs: String,
    @Json(name = "bookmarked") val bookmarked: Boolean,
)

coderabbitai · 2025-05-20T23:45:54Z

...e/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsCompleteViewModel.kt

+@HiltViewModel
+internal class InterestsCompleteViewModel @Inject constructor() : BaseViewModel<InterestsCompleteState, InterestsCompleteSideEffect>(InterestsCompleteState.getInitialState())


🛠️ Refactor suggestion

ViewModel에 학생 이름 설정 메서드 추가 필요

현재 ViewModel은 studentName 상태를 빈 문자열로 초기화하지만, 네비게이션 인자에서 받은 학생 이름을 설정하는 메서드가 없습니다. 네비게이션 흐름에 따라 이름을 설정할 수 있는 메서드가 필요합니다.

다음과 같이 ViewModel에 메서드를 추가하는 것을 권장합니다:

fun setStudentName(name: String) { setState { copy(studentName = name) } }

🤖 Prompt for AI Agents

In feature/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsCompleteViewModel.kt around lines 8 to 9, the ViewModel lacks a method to update the studentName state from navigation arguments. Add a public function named setStudentName that takes a String parameter and updates the state by copying the current state with studentName set to the provided name using setState.

feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsScreen.kt

coderabbitai · 2025-05-20T23:45:54Z

feature/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsViewmodel.kt

+    private fun fetchInterests() {
+        viewModelScope.launch(Dispatchers.IO) {
+            fetchInterestsUseCase().onSuccess {
+                setState {
+                    state.value.copy(
+                        studentName = it.studentName,
+                        interestsMajorList = it.interests,
+                    )
+                }
+            }
+        }


🛠️ Refactor suggestion

에러 처리 누락

fetchInterestsUseCase() 실패 시에도 UI 에서는 로딩 상태가 영구히 유지되거나 잘못된 데이터가 노출될 수 있습니다.
onFailure 블록을 추가하여 에러를 기록하고, 사용자에게 피드백(스낵바 등)을 제공하도록 해 주세요.

🤖 Prompt for AI Agents

In feature/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsViewmodel.kt around lines 50 to 60, the fetchInterests function lacks error handling for fetchInterestsUseCase failure. Add an onFailure block to log the error and update the UI state to stop loading and show an error message or snackbar to inform the user about the failure.

feature/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsViewmodel.kt

core/network/build.gradle.kts

feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsCompleteScreen.kt

feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsScreen.kt

feature/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsViewmodel.kt

coderabbitai

Actionable comments posted: 4

♻️ Duplicate comments (2)

feature/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsViewmodel.kt (2)

33-47: 에러 처리 누락

fetchCodeUseCase() 실패 시 UI에서는 로딩 상태가 영구히 유지되거나 잘못된 데이터가 노출될 수 있습니다.

49-60: 에러 처리 누락

fetchInterestsUseCase() 실패 시에도 UI에서는 로딩 상태가 영구히 유지되거나 잘못된 데이터가 노출될 수 있습니다.

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3ffa70b and 93772d6.

📒 Files selected for processing (4)

feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsCompleteScreen.kt (1 hunks)
feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsScreen.kt (2 hunks)
feature/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsViewmodel.kt (1 hunks)
feature/interests/src/main/res/values/strings.xml (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (3)

feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsCompleteScreen.kt
feature/interests/src/main/res/values/strings.xml
feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsScreen.kt

⏰ Context from checks skipped due to timeout of 90000ms (2)

GitHub Check: lint_and_build (build)
GitHub Check: lint_and_build (lint)

🔇 Additional comments (1)

feature/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsViewmodel.kt (1)

74-84: 결과 처리 잘 구현됨

과거 리뷰 피드백을 반영하여 성공 시 완료 화면 이동, 실패 시 에러 사이드이펙트 발행이 적절히 구현되었습니다.

feature/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsViewmodel.kt

coderabbitai · 2025-06-05T02:29:23Z

feature/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsViewmodel.kt

+    private fun fetchInterestsSearchRecruitments() {
+        viewModelScope.launch(Dispatchers.IO) {
+            fetchInterestsSearchRecruitmentsUseCase().onSuccess {
+                setState {
+                    state.value.copy(
+                        interestsRecruitments = it,
+                    )
+                }
+            }
+        }
+    }


⚠️ Potential issue

에러 처리 누락

fetchInterestsSearchRecruitmentsUseCase() 실패 시 에러 처리가 없어 사용자에게 적절한 피드백이 제공되지 않습니다.

private fun fetchInterestsSearchRecruitments() { viewModelScope.launch(Dispatchers.IO) { - fetchInterestsSearchRecruitmentsUseCase().onSuccess { + fetchInterestsSearchRecruitmentsUseCase() + .onSuccess { setState { state.value.copy( interestsRecruitments = it, ) } + }.onFailure { + // 에러 로깅 및 사용자 피드백 처리 } } }

📝 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

private fun fetchInterestsSearchRecruitments() {

viewModelScope.launch(Dispatchers.IO) {

fetchInterestsSearchRecruitmentsUseCase().onSuccess {

setState {

state.value.copy(

interestsRecruitments = it,

)

}

}

}

}

private fun fetchInterestsSearchRecruitments() {

viewModelScope.launch(Dispatchers.IO) {

fetchInterestsSearchRecruitmentsUseCase()

.onSuccess {

setState {

state.value.copy(

interestsRecruitments = it,

)

}

}.onFailure {

// 에러 로깅 및 사용자 피드백 처리

}

}

}

🤖 Prompt for AI Agents

In feature/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsViewmodel.kt around lines 62 to 72, the fetchInterestsSearchRecruitments function lacks error handling for the failure case of fetchInterestsSearchRecruitmentsUseCase(). Add an onFailure block to handle errors by updating the state or notifying the user appropriately, ensuring that any exceptions or failures are caught and proper feedback is provided.

coderabbitai · 2025-06-05T02:29:23Z

feature/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsViewmodel.kt

+@Immutable
+internal data class InterestsState(
+    val studentName: String,
+    val majorList: List<CodesEntity.CodeEntity>,
+    val interestsMajorList: List<InterestsEntity.InterestMajorEntity>,
+    val interestsRecruitments: InterestsRecruitmentsEntity?,
+    val selectedMajorCodes: List<Long>,
+    val selectedMajorCount: Int,
+    var buttonEnable: Boolean,
+) {
+    companion object {
+        fun getInitialState() = InterestsState(
+            studentName = "",
+            majorList = emptyList(),
+            interestsMajorList = emptyList(),
+            interestsRecruitments = null,
+            selectedMajorCodes = emptyList(),
+            selectedMajorCount = 0,
+            buttonEnable = false,
+        )
+    }
+}


🛠️ Refactor suggestion

buttonEnable 상태 업데이트 로직 누락

buttonEnable 프로퍼티가 정의되어 있지만 실제로 업데이트하는 로직이 없어 항상 false 상태로 유지됩니다. 선택된 관심사 개수에 따라 버튼 활성화 상태를 업데이트하는 로직을 추가해야 합니다.

setState { state.value.copy( selectedMajorCodes = currentSelected, selectedMajorCount = currentSelected.size, + buttonEnable = currentSelected.isNotEmpty(), ) }

Committable suggestion skipped: line range outside the PR's diff.

🤖 Prompt for AI Agents

In feature/interests/src/main/java/team/retum/jobis/interests/viewmodel/InterestsViewmodel.kt around lines 104 to 125, the buttonEnable property in InterestsState is never updated and remains false. Add logic to update buttonEnable based on the selectedMajorCount or selectedMajorCodes so that the button is enabled when there are selected interests. This update should occur whenever the selected majors change, ensuring buttonEnable reflects the current selection state.

coderabbitai

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0b2b6d8 and 335b3ba.

📒 Files selected for processing (1)

feature/landing/src/main/java/team/retum/landing/LandingViewModel.kt (1 hunks)

🧰 Additional context used

🪛 detekt (1.23.8)

feature/landing/src/main/java/team/retum/landing/LandingViewModel.kt

[warning] 3-4: The class or object LandingViewModel is empty.

(detekt.empty-blocks.EmptyClassBlock)

⏰ Context from checks skipped due to timeout of 90000ms (2)

GitHub Check: lint_and_build (lint)
GitHub Check: lint_and_build (build)

coderabbitai · 2025-06-13T13:58:05Z

feature/landing/src/main/java/team/retum/landing/LandingViewModel.kt

+class LandingViewModel {
+}


💡 Verification agent

🧩 Analysis chain

ViewModel 상속 및 DI 셋업 필요

현재 빈 클래스 블록으로 정의되어 있어 실제로 사용되지 않습니다. Android ViewModel을 상속하고, Hilt를 사용 중이라면 @HiltViewModel 어노테이션과 @Inject 생성자를 추가해 주세요.

구체적인 템플릿 코드 예시를 생성해 드릴까요?

🏁 Script executed:

#!/bin/bash set -e echo "=== Searching for HiltViewModel annotations ===" rg "@HiltViewModel" -n || echo "No @HiltViewModel annotations found" echo "=== Searching for Hilt AndroidApp setup ===" rg "@HiltAndroidApp" -n || echo "No @HiltAndroidApp annotations found" echo "=== Searching for dagger.hilt imports ===" rg "import dagger.hilt" -n || echo "No dagger.hilt imports found" echo "=== Searching for ViewModel imports ===" rg "import androidx.lifecycle.ViewModel" -n || echo "No androidx.lifecycle.ViewModel imports found" echo "=== Displaying LandingViewModel.kt snippet ===" head -n 20 feature/landing/src/main/java/team/retum/landing/LandingViewModel.kt || echo "LandingViewModel.kt not found"

Length of output: 11759

LandingViewModel에 ViewModel 상속 및 Hilt DI 설정 추가 필요

현재 feature/landing/src/main/java/team/retum/landing/LandingViewModel.kt 파일의 LandingViewModel 클래스가 빈 클래스(class LandingViewModel {})로 정의되어 있어 실제로 사용되지 않으며, 다른 ViewModel들과 동일한 패턴으로 동작하지 않습니다. 다음 사항을 반영해 주세요:

Android ViewModel 또는 공통 BaseViewModel을 상속받을 것

Hilt를 사용 중이므로 @HiltViewModel 어노테이션 추가

의존성 주입을 위해 @Inject constructor() 정의

예시:

package team.retum.landing import dagger.hilt.android.lifecycle.HiltViewModel import javax.inject.Inject import team.retum.common.base.BaseViewModel // 또는 androidx.lifecycle.ViewModel @HiltViewModel class LandingViewModel @Inject constructor( private val landingUseCase: LandingUseCase // 필요한 의존성으로 대체 ) : BaseViewModel() { // 또는 : ViewModel() // TODO: 기능 구현 }

의존성이 없을 경우 @Inject constructor()만 사용하셔도 무방합니다.

추가 예시가 필요하면 알려주세요.

🧰 Tools

🪛 detekt (1.23.8)

[warning] 3-4: The class or object LandingViewModel is empty.

(detekt.empty-blocks.EmptyClassBlock)

🤖 Prompt for AI Agents

In feature/landing/src/main/java/team/retum/landing/LandingViewModel.kt at lines 3 to 4, the LandingViewModel class is currently empty and does not extend ViewModel or set up dependency injection. To fix this, make LandingViewModel inherit from Android ViewModel or your BaseViewModel, add the @HiltViewModel annotation, and define an @Inject constructor for DI. If there are no dependencies yet, use an empty @Inject constructor; otherwise, inject required use cases or repositories as constructor parameters.

feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsCompleteScreen.kt

uson1004 added 30 commits April 24, 2025 23:17

feat :: 관심 분야 선택 viewmodel 생성

de1f6a3

refactor :: 관심 분야 페이지 네비게이션 등록

3be3909

chore :: 서버 url 추가

1a09dc2

feat :: 관심 분야 data class model 생성

1928a5f

feat :: 관심 분야 api 생성

5fecb10

refactor :: 관심 분야 네비게이션 등록

78e537d

refactor :: api model 이름 변경

36a12fa

feat :: 관심 분야 dataSource 구현

3e95ef4

feat :: 관심 분야 Repository 구현

3752a33

feat :: 관심 분야 UseCase 구현

5837000

refactor :: api 함수 이름 명칭 변경

4a3429c

feat :: 관심 분야 Entity 구현

a7dcde0

refactor :: Interests 모듈 namespace 변경

15a1c46

refactor :: data 모델 JsonClass 등록

c6351da

chore :: 관심분야 network 의존성 추가

72cc1dc

refactor :: 관심 분야 로직 이름 변경

6a30378

feat :: 관심 분야 의존성 주입

5b26033

feat :: 관심 분야 ViewModel 구현

192f26a

Merge remote-tracking branch 'origin/feature/462-관심-분야-선택-기능-구현' into…

7c86d38

… feature/462-관심-분야-선택-기능-구현

chore :: ktlint check

baa0c24

feat :: string 리소스 추가

024dd5e

refactor :: 관심 분야 UI 리팩토링

95cacff

feat :: 관심 분야 UI 퍼블리싱

2a52e81

build :: 프로젝트 namespace 변경

4917e85

refactor :: usecase entity 데이터로 변경

eeeba7d

refactor :: UseCase 이름 api 명칭과 동일하게 변경

393bfc7

refactor :: 관심 모집의뢰서 조회 entity로 매핑

6b7b6b8

refactor :: 관심 분야 설정 request 값 리스트로 변경

bfe0e4f

refactor :: 관심 분야 viewModel 기능 함수 구현

1603aad

refactor :: 관심 모집의뢰서 조회 UseCase 이름 변경

56fa277

uson1004 added the 기능 구현 비즈니스 로직을 작성합니다. label May 20, 2025

uson1004 linked an issue May 20, 2025 that may be closed by this pull request

관심 분야 선택 기능 구현 #462

Closed

1 task

github-project-automation bot added this to return 3기 backlog May 20, 2025

github-project-automation bot moved this to Todo in return 3기 backlog May 20, 2025

coderabbitai bot reviewed May 20, 2025

View reviewed changes

uson1004 added 2 commits May 21, 2025 08:57

chore :: ktlint refactoring

d8ceb92

chore :: ktlint change

3ffa70b

uson1004 changed the title ~~Feature/462 관심 분야 선택 기능 구현~~ 🔗 :: (#462) 관심 분야 선택 기능 구현 Jun 4, 2025

parkuiery reviewed Jun 4, 2025

View reviewed changes

uson1004 added 5 commits June 5, 2025 10:12

refactor :: 이전에 선택된 전공도 해제되도록 변경

050d321

fix :: dev url 숨김

b565488

refactor :: string value 적용

964bbfa

refator :: 상태 변경 값 viewmodel로 구현

a8d7c8f

refactor :: 관심분야 등록 함수 예외처리

93772d6

coderabbitai bot reviewed Jun 5, 2025

View reviewed changes

uson1004 added 2 commits June 5, 2025 12:10

refactor :: viewmodel에서 interests button 상태 변경

b78bdb5

refactor :: UI 로직 함수 UI 스레드에서 작업하도록 변경

0b2b6d8

uson1004 requested a review from parkuiery June 6, 2025 07:38

chore :: hilt compile error로 인한 파일 생성

335b3ba

coderabbitai bot reviewed Jun 13, 2025

View reviewed changes

chore :: ktlint change

6e5f4b6

uson1004 moved this from Todo to In review in return 3기 backlog Jun 16, 2025

uson1004 added 2 commits June 30, 2025 19:56

refactor :: column으로 관심분야 중앙 정렬

68147b9

chore :: ktlint check

ab502e2

parkuiery reviewed Jul 2, 2025

View reviewed changes

feature/interests/src/main/java/team/retum/jobis/interests/ui/InterestsCompleteScreen.kt Outdated Show resolved Hide resolved

parkuiery approved these changes Jul 2, 2025

View reviewed changes

uson1004 merged commit afdcce9 into develop Jul 2, 2025
2 checks passed

github-project-automation bot moved this from In review to Done in return 3기 backlog Jul 2, 2025

uson1004 deleted the feature/462-관심-분야-선택-기능-구현 branch July 31, 2025 14:18

		@HiltViewModel
		internal class InterestsCompleteViewModel @Inject constructor() : BaseViewModel<InterestsCompleteState, InterestsCompleteSideEffect>(InterestsCompleteState.getInitialState())

🔗 :: (#462) 관심 분야 선택 기능 구현 #465

🔗 :: (#462) 관심 분야 선택 기능 구현 #465

Uh oh!

Conversation

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

Uh oh!

개요

작업 내용

할 말

Summary by CodeRabbit

Uh oh!

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

Uh oh!

Walkthrough

Changes

Sequence Diagram(s)

Suggested reviewers

Poem

Chat

Support

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (.coderabbit.yaml)

Documentation and Community

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot May 20, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

coderabbitai bot May 20, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

coderabbitai bot Jun 5, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot Jun 5, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot Jun 13, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

uson1004 commented May 20, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented May 20, 2025 •

edited

Loading

CodeRabbit Configuration File (`.coderabbit.yaml`)