🔗 :: (#448) 취업률 상세 조회 페이지 제작 #449

uson1004 · 2025-03-05T11:08:01Z

개요

작업 내용

상세 취업률 페이지를 개발하였습니다
총 학생 취업률 화면 퍼블리싱
splits api 비활성화
employment 모듈 추가
ResourseKey 추가
서버 로직 작성
이미지 추가

할 말

부족한 점이 많이 피드백 부탁드립니다

Summary by CodeRabbit

새로운 기능
- 채용(취업) 모듈이 추가되어 사용자가 채용 현황 및 통계 정보를 확인할 수 있습니다.
- 앱 내 네비게이션이 개선되어 홈 화면에서 채용 관련 화면으로 손쉽게 이동할 수 있습니다.
- 채용 세부 화면에서는 회사 로고와 관련 데이터를 시각적으로 제공해 직관적인 정보를 전달합니다.
- 채용 통계 정보를 표시하는 UI가 추가되어 사용자가 취업률, 총 학생 수, 합격 학생 수를 쉽게 확인할 수 있습니다.
- 한국어로 된 채용 관련 상태 메시지가 추가되어 다국어 지원이 향상되었습니다.
Chores
- 프로젝트 의존성, 빌드 설정, Gradle 스크립트 등이 업데이트되어 전반적인 안정성과 성능이 향상되었습니다.

ksp 버전 호환

coderabbitai · 2025-03-05T11:08:08Z

Walkthrough

이번 PR에서는 고용(employment) 기능의 도입과 관련된 여러 변경 사항이 포함되어 있습니다. 앱 모듈에서 고용 관련 네비게이션 기능 및 의존성이 추가되었으며, Core 계층에서는 고용 상태를 가져오는 API와 데이터 모델, 리포지토리, 유즈케이스 인터페이스가 확장되었습니다. 또한, 고용 기능 전용 모듈이 새롭게 추가되어 UI, 네비게이션, 뷰모델 및 리소스 등이 정의되었고, Gradle 구성 파일과 빌드 스크립트에서도 업데이트가 이루어졌습니다.

Changes

파일	변경 요약
앱 및 홈 모듈 `app/build.gradle.kts`, `app/.../JobisNavigator.kt`, `app/.../MainNavigation.kt`, `app/.../RootNavigation.kt`, `app/.../RootScreen.kt`, `feature/home/.../HomeNavigation.kt`, `feature/home/.../HomeScreen.kt`	Debug 빌드 설정의 일부 주석 처리 및 고용 의존성 추가, 고용 네비게이션 기능 (onEmploymentClick 콜백) 통합
Core 모듈 `core/common/.../ResourceKeys.kt` `core/data/.../ApplicationRepository.kt`, `core/data/.../ApplicationRepositoryImpl.kt` `core/domain/.../EmploymentStatusEntity.kt`, `core/domain/.../FetchEmploymentStatusUseCase.kt` `core/network/.../ApplicationApi.kt`, `core/network/.../RemoteApplicationDataSource.kt`, `core/network/.../RemoteApplicationDataSourceImpl.kt`, `core/network/.../RequestUrls.kt`, `core/network/.../FetchEmploymentStatusResponse.kt`	고용 상태를 가져오는 API, 저장소 인터페이스 및 구현, 엔티티와 확장 함수 추가로 고용 상태 기능 구현
Employment Feature 모듈 `feature/employment/.gitignore`, `feature/employment/build.gradle.kts`, `feature/employment/src/main/AndroidManifest.xml`, `feature/employment/src/main/java/.../CompanyItem.kt`, `feature/employment/src/main/java/.../EmploymentNavigation.kt`, `feature/employment/src/main/java/.../EmploymentDetailScreen.kt`, `feature/employment/src/main/java/.../EmploymentScreen.kt`, `feature/employment/src/main/java/.../EmploymentDetailViewModel.kt`, `feature/employment/src/main/java/.../EmploymentViewModel.kt`, `feature/employment/src/main/res/values/strings.xml`	고용 기능 전용 모듈 추가: UI 구성요소, 네비게이션, 데이터 모델, 뷰모델, 문자열 리소스 등 전반적인 기능 도입
Gradle 및 스크립트 `gradle/libs.versions.toml`, `gradle/wrapper/gradle-wrapper.properties`, `gradlew`, `gradlew.bat`, `settings.gradle.kts`	Gradle 플러그인 및 라이브러리 버전 업데이트, Gradle 래퍼 버전 변경, 스크립트 개선 및 모듈 설정에 고용 모듈 추가

Sequence Diagram(s)

sequenceDiagram
    participant User
    participant HomeScreen
    participant JobisNavigator
    participant NavController
    participant EmploymentScreen

    User->>HomeScreen: 고용 버튼 클릭
    HomeScreen->>JobisNavigator: onEmploymentClick 콜백 호출
    JobisNavigator->>NavController: navigateToEmployment() 호출
    NavController->>EmploymentScreen: 고용 화면으로 이동

sequenceDiagram
    participant EmploymentViewModel
    participant FetchEmploymentStatusUseCase
    participant ApplicationRepository
    participant NetworkLayer

    EmploymentViewModel->>FetchEmploymentStatusUseCase: invoke() 호출
    FetchEmploymentStatusUseCase->>ApplicationRepository: fetchEmploymentStatus() 요청
    ApplicationRepository->>NetworkLayer: 고용 상태 요청
    NetworkLayer-->>ApplicationRepository: FetchEmploymentStatusResponse 반환
    ApplicationRepository-->>FetchEmploymentStatusUseCase: 응답 전달
    FetchEmploymentStatusUseCase-->>EmploymentViewModel: 엔티티 변환 후 반환

Suggested reviewers

Tmdhoon2

Poem

나는 귀여운 토끼, 코를 킁킁
코 앞에 새로운 기능이 반짝반짝
고용의 길 열어주는 네비게이터
코드 숲을 뛰어다니며 희망을 노래해
"Hop hop!"하며 앞으로 달려가네 🐰✨

✨ Finishing Touches

📝 Generate Docstrings

🪧 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.
- Generate unit testing code for this file.
- 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 generate unit testing code for this file.
- @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 generate unit testing code.
- @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.

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 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: 15

🧹 Nitpick comments (17)

app/build.gradle.kts (1)
56-59: 주석 처리된 디버그 구성을 정리하세요.

주석 처리된 구성 설정(ABI 분할, AAPT 크런치, 밀도 분할)이 있습니다. 이러한 주석을 유지하는 이유가 있다면 주석으로 설명을 추가하거나, 불필요하다면 완전히 제거하는 것이 코드 가독성을 높일 수 있습니다.
-//            splits.abi.isEnable = false
-//            aaptOptions.cruncherEnabled = false
-//            splits.density.isEnable = false
-//            aaptOptions.cruncherEnabled = false
core/network/src/main/java/team/retum/network/api/ApplicationApi.kt (1)
39-40: API 함수에 문서 주석을 추가하는 것이 좋습니다.

고용 상태 정보를 가져오는 API 함수가 추가되었지만, 이 함수의 목적과 반환값에 대한 설명이 없습니다. 코드 유지보수와 이해를 위해 KDoc 형식의 주석을 추가하는 것이 좋습니다.
+    /**
+     * 취업 현황 정보를 가져옵니다.
+     * 
+     * @return 취업 상태 정보가 포함된 응답 객체
+     */
     @GET(RequestUrls.Applications.employment)
     suspend fun fetchEmploymentStatus(): FetchEmploymentStatusResponse
core/network/src/main/java/team/retum/network/datasource/application/RemoteApplicationDataSource.kt (1)
26-26: 데이터 소스 함수에 문서 주석을 추가하세요.

고용 상태 정보를 가져오는 함수가 인터페이스에 추가되었지만, 이 함수의 목적과 반환값에 대한 설명이 없습니다. 코드 유지보수와 이해를 위해 KDoc 형식의 주석을 추가하는 것이 좋습니다.
+    /**
+     * 취업 현황 정보를 가져옵니다.
+     * 
+     * @return 취업 상태 정보가 포함된 응답 객체
+     */
     suspend fun fetchEmploymentStatus(): FetchEmploymentStatusResponse
core/network/src/main/java/team/retum/network/datasource/application/RemoteApplicationDataSourceImpl.kt (1)
21-25: 코드 중복이 발생하고 있습니다.

현재 각 fetch 메서드마다 RequestHandler 호출 패턴이 반복되고 있습니다. 이러한 반복 패턴은 코드 중복으로 이어질 수 있습니다.

중복 코드를 줄이기 위해 공통 fetch 로직을 추출하는 것을 고려해보세요:
+ private suspend fun <T> fetch(request: suspend () -> T): T {
+     return RequestHandler<T>().request {
+         request()
+     }
+ }

  override suspend fun fetchEmploymentCount(): FetchEmploymentCountResponse {
-     return RequestHandler<FetchEmploymentCountResponse>().request {
-         applicationApi.fetchEmploymentCount()
-     }
+     return fetch { applicationApi.fetchEmploymentCount() }
  }

  override suspend fun fetchEmploymentStatus(): FetchEmploymentStatusResponse {
-     return RequestHandler<FetchEmploymentStatusResponse>().request {
-         applicationApi.fetchEmploymentStatus()
-     }
+     return fetch { applicationApi.fetchEmploymentStatus() }
  }
이러한 패턴을 모든 fetch 메서드에 적용하면 코드의 중복을 줄이고 향후 유지보수를 더 쉽게 할 수 있습니다.

Also applies to: 57-61
gradlew.bat (1)

28-29: 배치 스크립트 주석 관련 조언

여기에 추가된 주석이 있습니다. 이 주석은 코드의 가독성을 높이지만, 쓰이지 않는 변수라면 이 주석이 실제로 필요한지 검토하는 것이 좋겠습니다.
core/network/src/main/java/team/retum/network/model/response/application/FetchEmploymentStatusResponse.kt (1)
6-24: 응답 모델 클래스가 잘 구현되었습니다.

백엔드 API 응답을 받기 위한 데이터 클래스 구조가 적절하게 구현되었습니다. 중첩 클래스를 사용하여 계층적 데이터 구조를 잘 표현하고 있으며, JSON 어노테이션도 적절히 적용되었습니다.

다만, 추후 확장성을 고려하여 내부 클래스를 별도 파일로 분리하는 것도 고려해 볼 수 있습니다.

내부 클래스를 별도 파일로 분리하는 방법:
// GetEmploymentRateList.kt
@JsonClass(generateAdapter = true)
data class GetEmploymentRateList(
    @Json(name = "id") val id: Int,
    @Json(name = "company_name") val companyName: String,
    @Json(name = "logo_url") val logoUrl: String
)

// ClassEmploymentStatusResponse.kt
@JsonClass(generateAdapter = true)
data class ClassEmploymentStatusResponse(
    @Json(name = "class_id") val classId: Int,
    @Json(name = "employment_rate_response_list") val employmentRateList: List<GetEmploymentRateList>,
    @Json(name = "total_students") val totalStudents: Int,
    @Json(name = "passed_students") val passedStudents: Int
)

// FetchEmploymentStatusResponse.kt
@JsonClass(generateAdapter = true)
data class FetchEmploymentStatusResponse(
    @Json(name = "classes") val classes: List<ClassEmploymentStatusResponse>
)
feature/home/src/main/java/team/retum/home/ui/HomeScreen.kt (2)
323-331: 문자열 리소스화가 필요합니다.

현재 버튼의 텍스트가 하드코딩되어 있습니다. 다국어 지원 및 일관성을 위해 문자열 리소스를 사용하는 것이 좋습니다.
-                text = "현황 보러 가기 ->",
+                text = stringResource(id = R.string.view_employment_status),
strings.xml 파일에 다음 리소스를 추가해주세요:
<string name="view_employment_status">현황 보러 가기 -></string>
323-331: 버튼 스타일 및 레이아웃 개선이 필요합니다.

현재 버튼의 스타일링이 다소 복잡합니다. wrapContentSize()와 fillMaxWidth(0.4f)를 함께 사용하는 것은 일관된 UI를 보장하지 않을 수 있습니다. 더 간단하고 명확한 방식으로 스타일을 적용하는 것이 좋습니다.
-            JobisSmallButton(
-                modifier = Modifier
-                    .wrapContentSize()
-                    .background(Color.Transparent)
-                    .fillMaxWidth(0.4f),
-                text = "현황 보러 가기 ->",
-                color = ButtonColor.Primary,
-                onClick = onEmploymentClick,
-            )
+            JobisSmallButton(
+                modifier = Modifier
+                    .background(Color.Transparent),
+                text = stringResource(id = R.string.view_employment_status),
+                color = ButtonColor.Primary,
+                onClick = onEmploymentClick,
+            )
core/domain/src/main/java/team/retum/usecase/entity/application/EmploymentStatusEntity.kt (2)
23-24: TODO 주석 제거 필요

코드베이스의 가독성과 프로페셔널리즘을 위해 이 TODO 주석을 제거하거나 실제 작업으로 처리해주세요.
-    classes = this.classes.map { it.toEntity() } // TODO :: 이 구조 이해 필요
+    classes = this.classes.map { it.toEntity() }
15-19: 데이터 클래스 네이밍 개선 제안

GetEmploymentRateList라는 이름에는 "Get" 접두사가 포함되어 있어 일반적인 엔티티 네이밍 컨벤션과 일치하지 않습니다. 다른 엔티티 클래스들과 일관성을 유지하기 위해 "Get" 접두사를 제거하는 것이 좋겠습니다.
-        data class GetEmploymentRateList(
+        data class EmploymentRateEntity(
             val id: Int,
             val companyName: String,
             val logoUrl: String,
         )
이후에 아래 코드와 같은 관련 참조들도 업데이트해야 합니다:

30줄: employmentRateList = this.employmentRateList.map { it.toEntity() },

35-39줄: 확장 함수를 업데이트
feature/employment/src/main/java/team/retum/employment/viewmodel/EmploymentViewModel.kt (2)
49-52: 한국어 주석은 영어로 번역 또는 통일 필요

코드베이스의 일관성을 위해 한국어 주석("익셉션의 집합")을 영어로 번역하거나, 프로젝트 전체에서 사용하는 주석 언어를 통일하는 것이 좋습니다.
-sealed interface EmploymentSideEffect { // 익셉션의 집합
+sealed interface EmploymentSideEffect { // Collection of exceptions/side effects
18-29: 소수점 표시 로직 개선 필요

현재 소수점 표시를 위해 DecimalFormat("##.#").format(rate)를 사용하고 있습니다. 이는 한글 로케일에서는 소수점이 쉼표(,)로 표시될 수 있습니다. 로케일에 관계없이 일관된 표시를 위해 명시적으로 로케일을 지정하는 것이 좋습니다.
-                        rate = DecimalFormat("##.#").format(rate),
+                        rate = DecimalFormat("##.#").apply { 
+                            DecimalFormatSymbols.getInstance(Locale.US).also { symbols ->
+                                decimalFormatSymbols = symbols
+                            }
+                        }.format(rate),
이를 위해 임포트 추가가 필요합니다:
import java.text.DecimalFormatSymbols
import java.util.Locale
feature/employment/src/main/java/team/retum/employment/ui/EmploymentScreen.kt (5)
60-60: TODO 주석 제거 필요

코드베이스의 가독성과 프로페셔널리즘을 위해 TODO 주석을 제거하거나 실제 작업으로 처리해주세요.
-) { // TODO :: 네비게이션 추가 및 부수
+) {
61-71: 애니메이션 논리 개선 필요

현재 animatedValue는 animateTo를 사용하여 이미 같은 값(rate)으로 애니메이션되고 있습니다. 또한, UI에 이 애니메이션 값이 사용되지 않고 있습니다. 이것은 불필요한 애니메이션 계산을 초래할 수 있습니다.
-    val animatedValue = remember { Animatable(rate) }
-    LaunchedEffect(Unit) {
-        animatedValue.animateTo(
-            targetValue = rate,
-            animationSpec = tween(durationMillis = 1000, easing = LinearEasing)
-        )
-        with(employmentViewModel) {
-            fetchEmploymentCount()
-        }
-    }
+    LaunchedEffect(Unit) {
+        employmentViewModel.fetchEmploymentCount()
+    }
또한 CircleProgress 컴포넌트는 자체 애니메이션 로직을 가지고 있으므로 여기서 애니메이션할 필요가 없습니다.

89-89: TODO 주석 제거 필요

코드베이스의 가독성과 프로페셔널리즘을 위해 TODO 주석을 제거하거나 실제 작업으로 처리해주세요.
-) { // TODO :: UI 퍼블리싱
+) {
252-308: CircleProgress 컴포넌트 개선 제안

CircleProgress 컴포넌트에 몇 가지 개선 사항이 있습니다:

StrokeCap.Butt와 StrokeCap.Round 불일치: 두 아크 사이에 스타일 불일치가 있어 시각적으로 일관성이 없을 수 있습니다.

중복 코드: 두 아크가 동일한 크기와 위치를 사용하므로 이를 공유 변수로 추출할 수 있습니다.

scaleX = -1f는 코드 가독성에 문제를 일으킬 수 있으며, 이보다는 startAngle과 sweepAngle을 조정하는 것이 더 명확합니다.
 Canvas(
     modifier = Modifier
         .size(radius)
-        .scale(scaleX = -1f, scaleY = 1f)
 ) {
     val canvasSize = size.minDimension
     val arcRadius = canvasSize / 2 - strokeWidth.toPx() / 2
+    val arcSize = Size(arcRadius * 2, arcRadius * 2)
+    val arcOffset = Offset(
+        (size.width - arcRadius * 2) / 2,
+        (size.height - arcRadius * 2) / 2
+    )

     drawArc(
         color = primaryColor,
-        startAngle = 270f,
-        sweepAngle = 360f,
+        startAngle = 90f,
+        sweepAngle = 360f,
         useCenter = false,
         style = Stroke(width = strokeWidth.toPx(), cap = StrokeCap.Round),
-        size = Size(arcRadius * 2, arcRadius * 2),
-        topLeft = Offset(
-            (size.width - arcRadius * 2) / 2,
-            (size.height - arcRadius * 2) / 2
-        )
+        size = arcSize,
+        topLeft = arcOffset
     )

     drawArc(
         color = secondaryColor,
-        startAngle = 270f,
-        sweepAngle = animatedValue.value * 3.6f,
+        startAngle = 90f,
+        sweepAngle = animatedValue.value * 3.6f,
         useCenter = false,
-        style = Stroke(width = strokeWidth.toPx(), cap = StrokeCap.Butt),
-        size = Size(arcRadius * 2, arcRadius * 2),
-        topLeft = Offset(
-            (size.width - arcRadius * 2) / 2,
-            (size.height - arcRadius * 2) / 2
-        )
+        style = Stroke(width = strokeWidth.toPx(), cap = StrokeCap.Round),
+        size = arcSize,
+        topLeft = arcOffset
     )
 }
310-363: ClassEmploymentButton UI 구조 개선 제안

ClassEmploymentButton 컴포넌트에는 중첩된 Surface와 Column이 많이 있어 불필요한 복잡성을 초래합니다. 또한, 여러 중첩된 패딩 구성이 있으며 일부는 중복됩니다.

중첩된 Surface 및 Column 구조를 단순화하고

패딩 값을 통합/정리하며

contentAlignment를 사용하여 레이아웃을 더 명확하게 표현하는 방향으로 리팩토링할 수 있습니다.
 @Composable
 fun ClassEmploymentButton(
     onClassClick: () -> Unit,
     @DrawableRes image: Int,
     description: String = "",
     text: String,
 ) {
     Surface(
-        modifier = Modifier.padding(),
+        modifier = Modifier,
         onClick = onClassClick,
         color = JobisTheme.colors.inverseSurface,
         shape = RoundedCornerShape(8.dp),
     ) {
         Column(
-            modifier = Modifier.background(color = JobisTheme.colors.inverseSurface)
+            modifier = Modifier
+                .padding(horizontal = 34.dp, vertical = 20.dp),
+            horizontalAlignment = Alignment.CenterHorizontally
         ) {
-            Column(
-                modifier = Modifier.padding(top = 20.dp, start = 34.dp, end = 34.dp),
-                verticalArrangement = Arrangement.SpaceBetween,
-                horizontalAlignment = Alignment.CenterHorizontally
-            ) {
-                Box(
-                    modifier = Modifier
-                        .size(80.dp)
-                        .clip(CircleShape)
-                        .background(JobisTheme.colors.background),
-                    contentAlignment = Alignment.Center
-                ) {
-                    Icon(
-                        modifier = Modifier
-                            .size(48.dp),
-                        painter = painterResource(image),
-                        contentDescription = description,
-                        tint = Color.Unspecified
-                    )
-                }
+            Box(
+                modifier = Modifier
+                    .size(80.dp)
+                    .clip(CircleShape)
+                    .background(JobisTheme.colors.background),
+                contentAlignment = Alignment.Center
+            ) {
+                Icon(
+                    modifier = Modifier
+                        .size(48.dp),
+                    painter = painterResource(image),
+                    contentDescription = description,
+                    tint = Color.Unspecified
+                )
             }
+            Spacer(modifier = Modifier.height(16.dp))
             Surface(
                 color = JobisTheme.colors.surfaceTint,
                 shape = RoundedCornerShape(12.dp),
                 modifier = Modifier
-                    .padding(top = 16.dp, start = 12.dp, bottom = 12.dp),
+                    .padding(start = 12.dp),
             ) {
                 JobisText(
                     text = text,
                     style = JobisTypography.SubBody,
                     color = JobisTheme.colors.background,
                     modifier = Modifier
                         .padding(horizontal = 16.dp, vertical = 8.dp)
                 )
             }
         }
     }
 }

📜 Review details

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

📥 Commits

Reviewing files that changed from the base of the PR and between 1734fe8 and 597e908.

⛔ Files ignored due to path filters (4)

core/design-system/src/main/res/drawable/ic_computer.png is excluded by !**/*.png
core/design-system/src/main/res/drawable/ic_robot.png is excluded by !**/*.png
core/design-system/src/main/res/drawable/ic_spanner.png is excluded by !**/*.png
gradle/wrapper/gradle-wrapper.jar is excluded by !**/*.jar

📒 Files selected for processing (32)

app/build.gradle.kts (2 hunks)
app/src/main/java/team/retum/jobisandroidv2/JobisNavigator.kt (2 hunks)
app/src/main/java/team/retum/jobisandroidv2/navigation/MainNavigation.kt (3 hunks)
app/src/main/java/team/retum/jobisandroidv2/root/RootNavigation.kt (2 hunks)
app/src/main/java/team/retum/jobisandroidv2/root/RootScreen.kt (4 hunks)
core/common/src/main/java/team/retum/common/utils/ResourceKeys.kt (2 hunks)
core/data/src/main/java/team/retum/data/repository/application/ApplicationRepository.kt (2 hunks)
core/data/src/main/java/team/retum/data/repository/application/ApplicationRepositoryImpl.kt (2 hunks)
core/domain/src/main/java/team/retum/usecase/entity/application/EmploymentStatusEntity.kt (1 hunks)
core/domain/src/main/java/team/retum/usecase/usecase/application/FetchEmploymentStatusUseCase.kt (1 hunks)
core/network/src/main/java/team/retum/network/api/ApplicationApi.kt (2 hunks)
core/network/src/main/java/team/retum/network/datasource/application/RemoteApplicationDataSource.kt (2 hunks)
core/network/src/main/java/team/retum/network/datasource/application/RemoteApplicationDataSourceImpl.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/response/application/FetchEmploymentStatusResponse.kt (1 hunks)
feature/employment/.gitignore (1 hunks)
feature/employment/build.gradle.kts (1 hunks)
feature/employment/src/main/AndroidManifest.xml (1 hunks)
feature/employment/src/main/java/team/retum/employment/model/CompanyItem.kt (1 hunks)
feature/employment/src/main/java/team/retum/employment/navigation/EmploymentNavigation.kt (1 hunks)
feature/employment/src/main/java/team/retum/employment/ui/EmploymentDetailScreen.kt (1 hunks)
feature/employment/src/main/java/team/retum/employment/ui/EmploymentScreen.kt (1 hunks)
feature/employment/src/main/java/team/retum/employment/viewmodel/EmploymentDetailViewModel.kt (1 hunks)
feature/employment/src/main/java/team/retum/employment/viewmodel/EmploymentViewModel.kt (1 hunks)
feature/employment/src/main/res/values/strings.xml (1 hunks)
feature/home/src/main/java/team/retum/home/navigation/HomeNavigation.kt (2 hunks)
feature/home/src/main/java/team/retum/home/ui/HomeScreen.kt (12 hunks)
gradle/libs.versions.toml (3 hunks)
gradle/wrapper/gradle-wrapper.properties (1 hunks)
gradlew (4 hunks)
gradlew.bat (4 hunks)
settings.gradle.kts (1 hunks)

✅ Files skipped from review due to trivial changes (4)

feature/employment/src/main/java/team/retum/employment/model/CompanyItem.kt
feature/employment/.gitignore
feature/employment/src/main/AndroidManifest.xml
feature/employment/src/main/res/values/strings.xml

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

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

🔇 Additional comments (56)

settings.gradle.kts (1)

46-46: 모듈 추가 확인됨

취업률 상세 조회 기능을 위한 :feature:employment 모듈이 프로젝트에 추가되었습니다. 모듈 추가가 잘 되었습니다.

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

71-71: API 엔드포인트 추가 확인됨

취업률 데이터를 가져오기 위한 API 엔드포인트 employment가 적절하게 추가되었습니다. 기존 패턴에 맞게 $path/employment 형식으로 정의되어 있어 일관성이 유지되었습니다.

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

13-13: 새로운 네비게이션 임포트 추가됨

취업률 화면으로 이동하기 위한 네비게이션 임포트가 적절하게 추가되었습니다.

138-140: 취업률 화면 네비게이션 함수 구현 확인

취업률 상세 화면으로 이동하는 navigateToEmployment() 함수가 기존 네비게이션 패턴에 맞게 잘 구현되어 있습니다. navController의 확장 함수를 적절히 호출하고 있습니다.

feature/employment/build.gradle.kts (4)

1-5: Gradle 플러그인 설정 확인

Android 라이브러리, Kotlin Android, KSP 플러그인이 적절하게 적용되었습니다. 필요한 플러그인들이 모두 포함되어 있습니다.

7-8: 공통 Gradle 플러그인 적용 확인

CommonGradlePlugin과 ComposeGradlePlugin이 적용되어 있어 프로젝트의 일관된 빌드 설정이 유지되고 있습니다.

10-17: Android 설정 확인

네임스페이스가 "team.retum.employment"로 올바르게 설정되어 있습니다. Java 호환성도 21 버전으로 적절히 설정되어 있습니다.

19-22: 종속성 확인

UI 툴링 미리보기 라이브러리가 추가되어 있습니다. 하지만 다른 필수 종속성이 일부 누락되어 있을 수 있습니다.

취업률 화면 구현에 필요한 core 모듈들(예: design-system, domain 등)에 대한 의존성이 추가되어 있는지 확인해 보세요. 일반적으로 feature 모듈은 기본적인 core 모듈들을 의존성으로 가져야 합니다.

app/build.gradle.kts (1)

117-117: 고용 기능 모듈 추가 확인됨.

고용률 상세 조회 페이지 제작을 위한 :feature:employment 모듈 의존성이 적절하게 추가되었습니다. 프로젝트 구조에 잘 통합되었습니다.

core/network/src/main/java/team/retum/network/api/ApplicationApi.kt (1)

12-12: ImportFetchEmploymentStatusResponse 추가 확인됨.

고용 상태 데이터를 처리하기 위한 응답 모델 import가 적절하게 추가되었습니다.

core/network/src/main/java/team/retum/network/datasource/application/RemoteApplicationDataSource.kt (1)

6-6: FetchEmploymentStatusResponse 임포트 추가 확인됨.

고용 상태 데이터를 처리하기 위한 응답 모델 import가 적절하게 추가되었습니다.

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

15-15: 고용 화면 탐색 파라미터 추가 확인됨.

취업률 상세 조회 페이지로 이동하기 위한 onEmploymentClick 파라미터가 적절하게 추가되었습니다. 다른 네비게이션 콜백과 일관성 있게 구현되었습니다.

39-39: Root 컴포저블에 onEmploymentClick 파라미터 전달 확인됨.

onEmploymentClick 콜백이 Root 컴포저블에 올바르게 전달되었습니다. 네비게이션 흐름이 적절하게 구현되었습니다.

core/data/src/main/java/team/retum/data/repository/application/ApplicationRepository.kt (2)

6-6: 필요한 import 문이 적절히 추가되었습니다.

네트워크 모델의 응답 클래스가 적절히 임포트되었습니다.

26-26: 신규 고용 상태 조회 메서드가 잘 정의되었습니다.

fetchEmploymentStatus 메서드가 인터페이스에 적절히 추가되었으며, 다른 fetch 메서드들과 동일한 패턴을 잘 따르고 있습니다. 비동기 작업을 위한 suspend 키워드도 올바르게 적용되었습니다.
gradle/wrapper/gradle-wrapper.properties (1)

3-5:
❓ Verification inconclusive

Gradle 버전 업데이트 및 네트워크 설정 추가에 대한 검토 필요

Gradle 버전을 8.2에서 8.9로 상당히 큰 폭으로 업데이트하였습니다. 이는 프로젝트의 빌드 프로세스에 영향을 미칠 수 있습니다. 또한 네트워크 타임아웃과 URL 검증 설정이 추가되었습니다.

이 변경 사항들이 프로젝트 요구사항에 맞는지, 다른 개발자들의 환경과 호환되는지 확인해주세요.

🏁 Script executed:
#!/bin/bash
# Gradle 버전 변경이 호환성 문제를 일으킬 수 있는지 확인
echo "Checking for compatibility issues with Gradle 8.9"

# build.gradle 파일에서 AGP 버전 확인
AGP_VERSION=$(grep -r "com.android.application" --include="*.gradle*" . | grep -o "version ['\"].*['\"]" | head -1 | cut -d"'" -f2 | cut -d'"' -f2)

if [ -n "$AGP_VERSION" ]; then
  echo "Android Gradle Plugin version found: $AGP_VERSION"
  
  # AGP 버전과 Gradle 8.9 호환성 체크 (간단한 예시)
  # 실제로는 더 복잡한 버전 호환성 로직이 필요할 수 있음
  VERSION_MAJOR=$(echo $AGP_VERSION | cut -d. -f1)
  
  if [ "$VERSION_MAJOR" -lt "8" ]; then
    echo "WARNING: AGP version $AGP_VERSION might not be fully compatible with Gradle 8.9"
  else
    echo "AGP version appears to be compatible with Gradle 8.9"
  fi
else
  echo "Could not determine Android Gradle Plugin version"
fi

# 프로젝트 내의 다른 플러그인 확인
echo "Checking other plugins in the project"
grep -r "apply plugin" --include="*.gradle*" .
Length of output: 527
[AGP 버전 확인 및 Gradle 8.9 업데이트 검증 요청]

Gradle 버전을 8.2에서 8.9로 업데이트하고, 네트워크 타임아웃(networkTimeout=10000)과 URL 검증(validateDistributionUrl=true) 설정이 추가되었습니다.

자동화된 스크립트 실행 결과, Android Gradle Plugin(AGP) 버전이 감지되지 않아 해당 플러그인이 실제로 사용 중인지 확인되지 않았습니다.

이 변경 사항이 프로젝트의 빌드 프로세스나 다른 개발자 환경에 미칠 영향을 방지하기 위해, AGP 및 기타 관련 플러그인들의 호환성을 수동으로 검토해주시기 바랍니다.
core/domain/src/main/java/team/retum/usecase/usecase/application/FetchEmploymentStatusUseCase.kt (1)

1-13: UseCase 구현이 잘 되었습니다.

FetchEmploymentStatusUseCase의 구현이 프로젝트의 다른 UseCase 패턴을 잘 따르고 있습니다:

ApplicationRepository 의존성 주입이 적절하게 이루어졌습니다.

suspend operator 함수를 사용하여 호출 시 간결한 문법을 제공합니다.

runCatching을 통해 예외 처리가 적절히 구현되었습니다.

응답 데이터를 엔티티로 변환하는 extension 함수 사용이 일관성 있게 적용되었습니다.

core/network/src/main/java/team/retum/network/datasource/application/RemoteApplicationDataSourceImpl.kt (2)

7-7: 필요한 import 문이 적절히 추가되었습니다.

새로운 기능에 필요한 네트워크 응답 모델 클래스가 잘 임포트되었습니다.

57-61: 고용 상태 조회 메서드가 일관된 패턴으로 구현되었습니다.

fetchEmploymentStatus 메서드가 다른 메서드들과 동일한 패턴을 따라 잘 구현되었습니다:

RequestHandler를 사용하여 네트워크 요청을 처리합니다.

적절한 API 엔드포인트(applicationApi.fetchEmploymentStatus())를 호출합니다.

인터페이스에 정의된 메서드를 올바르게 오버라이드하였습니다.

gradlew.bat (4)

17-17: Windows 배치 스크립트 업데이트에 대한 검토

이 부분은 기존 코드와 동일합니다. %DEBUG% 변수가 비어있는지 확인하고 비어있으면 echo를 꺼버립니다.

44-44: Windows 배치 스크립트 구문 개선

숫자 비교에 == 연산자 대신 equ 연산자를 사용하는 것은 Windows 배치 스크립트에서 더 정확한 접근 방식입니다. 이 변경은 스크립트의 오류 처리를 개선합니다.

79-79: Windows 배치 스크립트 구문 개선

여기서도 숫자 비교를 위해 == 연산자 대신 equ 연산자를 사용하도록 변경되었습니다. 이는 배치 스크립트의 오류 처리 부분을 개선합니다.

84-87: 향상된 오류 처리 로직

이 부분에서는 오류 처리 로직이 향상되었습니다:

ERRORLEVEL 값을 EXIT_CODE 변수에 저장

EXIT_CODE가 0이면 1로 설정 (오류 플래그)

GRADLE_EXIT_CONSOLE 변수 값에 따라 적절한 종료 코드로 스크립트 종료

이러한 변경은 스크립트의 오류 처리 메커니즘을 더 명확하고 강력하게 만듭니다.

core/data/src/main/java/team/retum/data/repository/application/ApplicationRepositoryImpl.kt (2)

7-7: 고용 상태 응답 모델 추가

새로운 기능에 필요한 FetchEmploymentStatusResponse 모델을 가져오는 import 문이 추가되었습니다. 이는 신규 기능 개발에 필요한 적절한 변경입니다.

46-48: 고용 상태 조회 기능 구현

새로운 고용 상태 조회 기능인 fetchEmploymentStatus() 메서드가 구현되었습니다. 이 메서드는 데이터 소스로부터 고용 상태 정보를 가져오는 단순 위임 패턴을 따르고 있어, 리포지토리 계층의 일관성을 유지하고 있습니다.

코드 구현이 깔끔하며 기존 메서드 구현 패턴을 잘 따르고 있습니다.

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

26-28: 고용 관련 리소스 키 추가

고용 상태 기능을 위한 새로운 상수들이 추가되었습니다:

RATE: 고용률 정보를 위한 키

TOTAL_STUDENT_COUNT: 전체 학생 수를 위한 키

PASS_COUNT: 합격 학생 수를 위한 키

이 상수들은 화면 간 데이터 전달이나 SharedPreferences 등에서 사용될 것으로 보입니다. 명확한 상수명을 사용한 좋은 구현입니다.

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

9-9: 고용 모듈 네비게이션 import 추가

새로 추가된 고용 관련 기능의 네비게이션을 위한 import 문이 추가되었습니다. 이는 신규 기능 개발에 필요한 적절한 변경입니다.

38-38: Root 네비게이션에 고용 기능 이동 콜백 추가

Root 네비게이션에 고용 화면으로 이동하기 위한 콜백 함수 onEmploymentClick이 추가되었습니다. 이 콜백은 navigator::navigateToEmployment와 연결되어 있어, 사용자가 고용 화면으로 이동할 수 있도록 합니다.

108-111: 고용 기능 네비게이션 추가

고용 화면을 위한 새로운 네비게이션 경로가 추가되었습니다. 이 경로는 다음 두 가지 콜백을 포함합니다:

onBackPressed: 뒤로 가기 버튼 처리를 위한 콜백

onClassClick: 특정 학급을 클릭했을 때 처리를 위한 콜백

새로운 기능을 위한 네비게이션 구성이 기존 패턴을 잘 따르고 있습니다.

feature/home/src/main/java/team/retum/home/navigation/HomeNavigation.kt (2)

15-15: 코드 구현이 잘 되었습니다.

새로운 onEmploymentClick 파라미터를 잘 추가하였습니다. 기존 코드 패턴을 따르고 있으며, 다른 클릭 핸들러와 일관성 있게 구현되었습니다.

27-27: Home 컴포저블에 onEmploymentClick 전달 구현이 적절합니다.

Home 컴포저블에 새로 추가된 onEmploymentClick 파라미터를 올바르게 전달하고 있습니다.

feature/home/src/main/java/team/retum/home/ui/HomeScreen.kt (9)

23-23: 새로운 레이아웃 속성 추가가 적절하게 이루어졌습니다.

wrapContentSize를 import하여 버튼의 크기 조정을 위해 사용한 것이 좋네요.

61-63: 버튼 컴포넌트 관련 import가 정확히 추가되었습니다.

필요한 ButtonColor 및 JobisSmallButton import가 적절하게 추가되었습니다.

83-83: API 일관성을 위한 매개변수 추가가 잘 되었습니다.

Home 함수에 onEmploymentClick 콜백 매개변수를 추가하여 UI와 내비게이션 로직이 잘 연결되었습니다.

130-131: HomeScreen 함수에 콜백 파라미터 전달이 올바르게 구현되었습니다.

HomeScreen에 onEmploymentClick 콜백을 전달하는 부분이 잘 구현되었습니다.

154-154: 콜백 함수 매개변수가 정확히 추가되었습니다.

HomeScreen 함수 시그니처에 onEmploymentClick 콜백이 올바르게 추가되었습니다.

186-187: Banner 컴포넌트에 콜백 함수가 적절히 전달되었습니다.

Banner 함수 호출 시 onEmploymentClick 콜백이 올바르게 전달되었습니다.

228-228: Banner 함수 시그니처에 콜백이 올바르게 추가되었습니다.

Banner 함수에 onEmploymentClick 매개변수가 적절히 추가되었습니다.

270-271: EmploymentRate 컴포넌트에 콜백이 잘 전달되었습니다.

EmploymentRate 호출 시 onEmploymentClick 콜백이 정확히 전달되었습니다.

309-309: 콜백 함수 매개변수가 적절히 추가되었습니다.

EmploymentRate 함수 시그니처에 onEmploymentClick 매개변수가 올바르게 추가되었습니다.
gradlew (7)

4-4: 저작권 정보가 업데이트되었습니다.

저작권 정보가 최신 연도로 업데이트되었습니다.

20-62: 스크립트 문서화 및 주석이 개선되었습니다.

Gradle 스크립트에 대한 문서화 및 설명이 상세하게 개선되었으며, 실행 및 패치 관련 중요 지침이 포함되었습니다.

70-81: 심볼릭 링크 처리 로직이 개선되었습니다.

expr 대신 쉘 파라미터 확장과 case 문을 사용하여 더 견고하고 가독성 있는 코드로 개선되었습니다.

133-140: Java 명령 검출 로직 개선이 이루어졌습니다.

which 대신 command -v를 사용하여 더 이식성 있는 방식으로 Java 명령을 검색하도록 개선되었습니다.

144-160: 파일 디스크립터 설정 로직이 개선되었습니다.

파일 디스크립터 최대값 설정 로직이 더 간결하고 명확하게 개선되었습니다.

211-247: 커맨드 라인 인수 처리 로직이 개선되었습니다.

JVM 옵션과 명령 인수 처리를 위한 로직이 더 견고하고 안전한 방식으로 개선되었으며, xargs를 활용한 인수 파싱이 추가되었습니다.

1-1:
❓ Verification inconclusive

셸 인터프리터 지정 방식 변경 검토가 필요합니다.

#!/usr/bin/env sh에서 #!/bin/sh로 변경되었습니다. 이는 시스템 간 호환성에 영향을 줄 수 있습니다. #!/usr/bin/env sh가 더 이식성이 높지만, 프로젝트의 요구사항에 따라 결정해주세요.

🏁 Script executed:
#!/bin/bash
# Check if /bin/sh is POSIX compliant
/bin/sh -c 'echo $POSIX2_VERSION'
Length of output: 36
검증 필요: 셸 인터프리터 지정 방식 변경 관련 추가 확인 요청
파일: gradlew (라인 1)

변경 내역: 기존의 #!/usr/bin/env sh에서 #!/bin/sh로 변경됨.

우려사항: /bin/sh가 POSIX 호환성을 충실히 제공하는지에 대한 검증 명령어(echo $POSIX2_VERSION) 실행 결과 아무 출력도 발생하지 않아, 해당 셸 인터프리터가 기대하는 동작을 보장하는지 불분명합니다.
권장 조치:
프로젝트 요구사항에 따라 이식성이 높은 /usr/bin/env sh 사용 여부를 재검토하시기 바랍니다.
추가 검증을 위해 아래와 같이 명령어를 수정하거나, 수동으로 POSIX 호환성 여부를 확인해 주시기 바랍니다.
#!/bin/bash
# POSIX2_VERSION 변수가 설정되어 있는지 명확히 확인하는 스크립트 예시
output=$(/bin/sh -c 'if [ -n "$POSIX2_VERSION" ]; then echo "POSIX compliant: $POSIX2_VERSION"; else echo "POSIX2_VERSION not set"; fi')
echo "POSIX check: $output"
위와 같이 결과가 명확하게 나오지 않는다면, 해당 변경이 시스템 간 호환성에 미치는 영향을 충분히 검토한 후 결정하시길 바랍니다.
app/src/main/java/team/retum/jobisandroidv2/root/RootScreen.kt (4)

49-49: Root 함수에 고용 현황 페이지 이동 콜백이 적절히 추가되었습니다.

onEmploymentClick 콜백 파라미터가 Root 함수에 올바르게 추가되었습니다.

83-83: RootScreen 호출에 콜백이 올바르게 전달되었습니다.

RootScreen 호출 시 onEmploymentClick 콜백이 적절히 전달되었습니다.

117-117: RootScreen 함수 시그니처에 콜백이 정확히 추가되었습니다.

RootScreen 함수에 onEmploymentClick 매개변수가 올바르게 추가되었습니다.

163-163: NavHost의 home 함수 호출에 콜백이 잘 전달되었습니다.

NavigationHost의 home 함수 호출 시 onEmploymentClick 콜백이 올바르게 전달되었습니다.

feature/employment/src/main/java/team/retum/employment/viewmodel/EmploymentDetailViewModel.kt (1)

11-13: ViewModel 초기화 및 의존성 주입이 잘 구현되었습니다.

ViewModel 클래스가 생성자를 통해 의존성 주입을 올바르게 받고 있으며, 기본 상태 초기화도 적절합니다.
core/domain/src/main/java/team/retum/usecase/entity/application/EmploymentStatusEntity.kt (1)

35-40:
❓ Verification inconclusive

URL 구성 로직 확인 필요

이미지 URL을 구성할 때 단순히 문자열을 연결하고 있습니다. 이렇게 하면 logoUrl에 이미 전체 URL이 포함되어 있거나 슬래시('/')가 누락된 경우 문제가 발생할 수 있습니다.

🏁 Script executed:
#!/bin/bash
# ResourceKeys.IMAGE_URL의 형식과 사용 방식 확인
cat $(fd ResourceKeys --type file) | grep -A 5 "IMAGE_URL"
Length of output: 463
URL 구성 로직 개선 필요

현재 EmploymentStatusEntity.ClassEmploymentStatusEntity.GetEmploymentRateList.toEntity()에서 이미지 URL을 구성할 때, ResourceKeys.IMAGE_URL 상수와 this.logoUrl를 단순 문자열 연결 방식으로 결합하고 있습니다. 이 방식은 아래와 같은 문제를 야기할 수 있습니다:

ResourceKeys.IMAGE_URL는 "https://dsm-s3-bucket-jobis.s3.ap-northeast-2.amazonaws.com/"와 같이 trailing slash가 포함되어 있으나,

this.logoUrl가 상대경로인 경우에는 문제가 없으나,

백엔드 응답에서 logoUrl에 이미 전체 URL이 포함된 경우 중복된 URL이 생길 수 있습니다.

this.logoUrl에 슬래시가 누락된 경우 올바른 URL 형식이 보장되지 않을 수 있습니다.

백엔드 응답 스펙과 전달되는 logoUrl 값의 형식을 재검토한 후, 조건부 로직(예: 전달된 URL이 절대 URL인지 체크 후 접두어를 적용하는 방식)을 적용하는 방향으로 개선을 검토해주시기 바랍니다.
gradle/libs.versions.toml (3)

51-54:
❓ Verification inconclusive

최신 라이브러리 버전 적합성 확인

이 PR에 추가된 라이브러리 버전이 최신 안정 버전인지 확인하는 것이 좋습니다:

core-ktx: 1.15.0

appcompat: 1.7.0

tiles-tooling-preview: 1.4.1

ui-tooling-preview-android: 1.7.8

🌐 Web query:
Android androidx.core:core-ktx 라이브러리의 최신 안정 버전은 무엇인가요?
Error during web search
최신 안정 버전 확인 필요

현재 gradle/libs.versions.toml 파일(라인 51-54)에 나열된 라이브러리 버전들이 최신 안정 버전인지 확인이 필요합니다.

core-ktx: 1.15.0

appcompat: 1.7.0

tiles-tooling-preview: 1.4.1

ui-tooling-preview-android: 1.7.8

웹 검색 시도 중 오류가 발생하여 최신 버전 정보를 확인할 수 없었습니다. 해당 라이브러리들의 최신 안정 버전을 수동으로 확인한 후, 필요한 경우 업데이트해주시기 바랍니다.

20-20:
❓ Verification inconclusive

안드로이드 Gradle 플러그인 버전 대폭 업데이트 검증 필요

Android Gradle 플러그인을 8.2.2에서 8.7.0으로 대폭 업데이트했습니다. 이는 주요 버전 변경으로, 호환성 문제나 빌드 설정 변경이 필요할 수 있습니다.

🌐 Web query:
Android Gradle Plugin 8.7.0과 8.2.2 사이의 주요 변경 사항 및 호환성 문제는 무엇인가요?
Error during web search
검증 필요: Android Gradle 플러그인 8.7.0 업데이트의 주요 변경 사항 확인

Android Gradle 플러그인을 8.2.2에서 8.7.0으로 업데이트하면서, major 버전 변경에 따른 다음 사항들을 확인해주세요:

릴리즈 노트 검토: AGP 8.7.0의 기능 추가, 제거 또는 변경된 API 및 구성 옵션 확인

빌드 설정: 기존의 빌드 설정 및 플러그인 호환성 문제 발생 여부 점검

호환성 테스트: 실제 프로젝트 빌드 및 테스트 과정에서 발생할 수 있는 문제 유무 확인

현재 웹 검색에서 관련 정보를 받지 못했으므로, 위 사항들을 추가 검증한 후 진행해 주세요.

120-123:
❓ Verification inconclusive

새로 추가된 라이브러리 사용 목적 명확화 필요

이 PR에서 새로 추가된 라이브러리들(core-ktx, appcompat, tiles-tooling-preview, ui-tooling-preview-android)의 구체적인 사용 목적과 필요성이 명확하지 않습니다. 특히 tiles-tooling-preview는 Wear OS 개발과 관련이 있는데, 이 프로젝트가 Wear OS 앱을 개발하는지 확인이 필요합니다.

🏁 Script executed:
#!/bin/bash
# Wear OS 관련 코드나 설정이 있는지 확인
fd -E "*.git*" -E "*build*" -E "*gradle*" -t f | xargs grep -l "wear" 2>/dev/null || echo "No Wear OS references found"

# 이 라이브러리들을 가져오는 모듈 확인
grep -r "androidx-tiles-tooling-preview\|androidx-core-ktx\|androidx-appcompat\|androidx-ui-tooling-preview-android" --include="*.gradle.kts" .
Length of output: 280
라이브러리 사용 목적 재검토 필요

PR에서 새로 추가된 라이브러리들(androidx-core-ktx, androidx-appcompat, androidx-tiles-tooling-preview, androidx-ui-tooling-preview-android)의 사용 용도가 코드상에서 명확하지 않습니다. 특히 androidx-tiles-tooling-preview는 Wear OS 개발에 활용되는 라이브러리로 보이나, 현재 코드베이스에서는 Wear OS와 관련된 참조가 발견되지 않았습니다.

파일: gradle/libs.versions.toml (라인: 120-123)

확인 요청: 해당 라이브러리들이 실제로 사용되는지, 혹은 앞으로 사용될지를 명확히 설명해 주시기 바랍니다. 만약 불필요하다면 정리하는 것도 고려해 주세요.

coderabbitai · 2025-03-05T11:13:35Z

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

@@ -1,5 +1,7 @@
 package team.retum.common.utils

+import android.provider.Telephony.Mms.Rate


🛠️ Refactor suggestion

불필요한 import 확인 필요

android.provider.Telephony.Mms.Rate import가 추가되었지만, 코드 내에서 사용되는 곳이 없는 것으로 보입니다. 사용되지 않는 import는 제거하는 것이 좋습니다.

-import android.provider.Telephony.Mms.Rate

📝 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

import android.provider.Telephony.Mms.Rate

coderabbitai · 2025-03-05T11:13:35Z

feature/employment/src/main/java/team/retum/employment/navigation/EmploymentNavigation.kt

+        route = "$NAVIGATION_EMPLOYMENT/${ResourceKeys}",
+        arguments = listOf(


⚠️ Potential issue

ResourceKeys 사용 방식에 오류가 있습니다.

현재 코드에서는 "$NAVIGATION_EMPLOYMENT/${ResourceKeys}"와 같이 ResourceKeys 객체를 직접 문자열에 삽입하려고 시도하고 있습니다. ResourceKeys는 클래스나 객체이므로 이렇게 사용하면 안 됩니다.

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

- route = "$NAVIGATION_EMPLOYMENT/${ResourceKeys}", + route = "$NAVIGATION_EMPLOYMENT/{${ResourceKeys.RATE}}/{${ResourceKeys.TOTAL_STUDENT_COUNT}}/{${ResourceKeys.PASS_COUNT}}",

📝 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

route = "$NAVIGATION_EMPLOYMENT/${ResourceKeys}",

arguments = listOf(

route = "$NAVIGATION_EMPLOYMENT/{${ResourceKeys.RATE}}/{${ResourceKeys.TOTAL_STUDENT_COUNT}}/{${ResourceKeys.PASS_COUNT}}",

arguments = listOf(

coderabbitai · 2025-03-05T11:13:35Z

feature/employment/src/main/java/team/retum/employment/navigation/EmploymentNavigation.kt

+fun NavController.navigateToEmployment() {
+    navigate(NAVIGATION_EMPLOYMENT)
+}


⚠️ Potential issue

navigateToEmployment 함수에 필요한 인수가 누락되었습니다.

navigateToEmployment 함수는 필요한 모든 인수(rate, totalStudentCount, passCount)를 포함하여 네비게이션 루트를 구성해야 합니다.

다음과 같이 수정하세요:

- fun NavController.navigateToEmployment() { - navigate(NAVIGATION_EMPLOYMENT) - } + fun NavController.navigateToEmployment(rate: Float, totalStudentCount: Long, passCount: Long) { + navigate("$NAVIGATION_EMPLOYMENT/$rate/$totalStudentCount/$passCount") + }

📝 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

fun NavController.navigateToEmployment() {

navigate(NAVIGATION_EMPLOYMENT)

}

fun NavController.navigateToEmployment(rate: Float, totalStudentCount: Long, passCount: Long) {

navigate("$NAVIGATION_EMPLOYMENT/$rate/$totalStudentCount/$passCount")

}

coderabbitai · 2025-03-05T11:13:36Z

feature/employment/src/main/java/team/retum/employment/navigation/EmploymentNavigation.kt

+        val rate = it.arguments?.getFloat(ResourceKeys.RATE) ?: 0f
+        val totalStudentCount = it.arguments?.getLong(ResourceKeys.RATE) ?: 0
+        val passCount = it.arguments?.getLong(ResourceKeys.RATE) ?: 0


⚠️ Potential issue

인수 키 참조에 오류가 있습니다.

세 개의 다른 인수를 가져오는데 모두 동일한 키(ResourceKeys.RATE)를 사용하고 있습니다. 각 인수에 맞는 키를 사용해야 합니다.

다음과 같이 수정하세요:

- val rate = it.arguments?.getFloat(ResourceKeys.RATE) ?: 0f - val totalStudentCount = it.arguments?.getLong(ResourceKeys.RATE) ?: 0 - val passCount = it.arguments?.getLong(ResourceKeys.RATE) ?: 0 + val rate = it.arguments?.getFloat(ResourceKeys.RATE) ?: 0f + val totalStudentCount = it.arguments?.getLong(ResourceKeys.TOTAL_STUDENT_COUNT) ?: 0 + val passCount = it.arguments?.getLong(ResourceKeys.PASS_COUNT) ?: 0

📝 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

val rate = it.arguments?.getFloat(ResourceKeys.RATE) ?: 0f

val totalStudentCount = it.arguments?.getLong(ResourceKeys.RATE) ?: 0

val passCount = it.arguments?.getLong(ResourceKeys.RATE) ?: 0

val rate = it.arguments?.getFloat(ResourceKeys.RATE) ?: 0f

val totalStudentCount = it.arguments?.getLong(ResourceKeys.TOTAL_STUDENT_COUNT) ?: 0

val passCount = it.arguments?.getLong(ResourceKeys.PASS_COUNT) ?: 0

coderabbitai · 2025-03-05T11:13:36Z

feature/employment/src/main/java/team/retum/employment/navigation/EmploymentNavigation.kt

+            navArgument(ResourceKeys.RATE) { NavType.FloatType},
+            navArgument(ResourceKeys.TOTAL_STUDENT_COUNT) { NavType.LongType},
+            navArgument(ResourceKeys.PASS_COUNT) { NavType.LongType }
+        )


🛠️ Refactor suggestion

네비게이션 인수 유형 지정이 불완전합니다.

네비게이션 인수의 유형은 지정되었지만, 각 인수의 이름에 대한 중괄호가 누락되어 있습니다.

다음과 같이 수정하세요:

- navArgument(ResourceKeys.RATE) { NavType.FloatType}, - navArgument(ResourceKeys.TOTAL_STUDENT_COUNT) { NavType.LongType}, - navArgument(ResourceKeys.PASS_COUNT) { NavType.LongType } + navArgument(ResourceKeys.RATE) { type = NavType.FloatType }, + navArgument(ResourceKeys.TOTAL_STUDENT_COUNT) { type = NavType.LongType }, + navArgument(ResourceKeys.PASS_COUNT) { type = NavType.LongType }

📝 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

navArgument(ResourceKeys.RATE) { NavType.FloatType},

navArgument(ResourceKeys.TOTAL_STUDENT_COUNT) { NavType.LongType},

navArgument(ResourceKeys.PASS_COUNT) { NavType.LongType }

)

navArgument(ResourceKeys.RATE) { type = NavType.FloatType },

navArgument(ResourceKeys.TOTAL_STUDENT_COUNT) { type = NavType.LongType },

navArgument(ResourceKeys.PASS_COUNT) { type = NavType.LongType }

)

coderabbitai · 2025-03-05T11:13:36Z

feature/employment/src/main/java/team/retum/employment/viewmodel/EmploymentDetailViewModel.kt

+data class EmploymentDetailState(
+    val classId: Long,
+    val totalStudent: Long,
+    val passStudent: Long,
+    val companyInfo: List<CompanyItem>
+) {
+    companion object {
+        fun getDefaultState() = EmploymentDetailState(
+            classId = 0,
+            totalStudent = 0,
+            passStudent = 0,
+            companyInfo = emptyList(),
+        )
+    }
+


🛠️ Refactor suggestion

클래스 구조 정보가 손실되고 있습니다.

현재 구현은 모든 클래스의 회사 정보를 하나의 리스트로 평면화(flatMap)하고 있어 원본 데이터의 클래스별 구조 정보가 손실됩니다. 클래스 구조를 보존하는 것이 좋습니다.

data class EmploymentDetailState( val classId: Long, val totalStudent: Long, val passStudent: Long, - val companyInfo: List<CompanyItem> + val classCompanyInfo: List<ClassCompanyInfo> ) { companion object { fun getDefaultState() = EmploymentDetailState( classId = 0, totalStudent = 0, passStudent = 0, - companyInfo = emptyList(), + classCompanyInfo = emptyList(), ) } } +data class ClassCompanyInfo( + val classId: Long, + val className: String, + val companies: List<CompanyItem> +)

그리고 fetchEmploymentStatus() 함수도 다음과 같이 수정해야 합니다:

internal fun fetchEmploymentStatus() { viewModelScope.launch(Dispatchers.IO) { fetchEmploymentStatusUseCase().onSuccess { setState { state.value.copy( - companyInfo = it.classes.flatMap { classInfo -> - classInfo.employmentRateList.map { employmentRate -> - CompanyItem( - companyName = employmentRate.companyName, - logoUrl = employmentRate.logoUrl - ) - } - } + classCompanyInfo = it.classes.map { classInfo -> + ClassCompanyInfo( + classId = classInfo.classId, + className = classInfo.className, + companies = classInfo.employmentRateList.map { employmentRate -> + CompanyItem( + companyName = employmentRate.companyName, + logoUrl = employmentRate.logoUrl + ) + } + ) + } ) }

📝 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

data class EmploymentDetailState(

val classId: Long,

val totalStudent: Long,

val passStudent: Long,

val companyInfo: List<CompanyItem>

) {

companion object {

fun getDefaultState() = EmploymentDetailState(

classId = 0,

totalStudent = 0,

passStudent = 0,

companyInfo = emptyList(),

)

}

data class EmploymentDetailState(

val classId: Long,

val totalStudent: Long,

val passStudent: Long,

val classCompanyInfo: List<ClassCompanyInfo>

) {

companion object {

fun getDefaultState() = EmploymentDetailState(

classId = 0,

totalStudent = 0,

passStudent = 0,

classCompanyInfo = emptyList(),

)

}

}

data class ClassCompanyInfo(

val classId: Long,

val className: String,

val companies: List<CompanyItem>

)

internal fun fetchEmploymentStatus() {

viewModelScope.launch(Dispatchers.IO) {

fetchEmploymentStatusUseCase().onSuccess {

setState {

state.value.copy(

classCompanyInfo = it.classes.map { classInfo ->

ClassCompanyInfo(

classId = classInfo.classId,

className = classInfo.className,

companies = classInfo.employmentRateList.map { employmentRate ->

CompanyItem(

companyName = employmentRate.companyName,

logoUrl = employmentRate.logoUrl

)

}

)

}

)

}

}

}

}

coderabbitai · 2025-03-05T11:13:37Z

feature/employment/src/main/java/team/retum/employment/viewmodel/EmploymentDetailViewModel.kt

+sealed interface EmploymentDetailSideEffect {
+    data object BadRequest : EmploymentDetailSideEffect
+    data object Success : EmploymentDetailSideEffect
+}


🛠️ Refactor suggestion

SideEffect 구현이 완성되지 않았습니다.

현재 EmploymentDetailSideEffect는 정의되어 있지만 실제로는 사용되지 않고 있습니다. ViewModel에서 적절한 시점에 side effect를 발행하도록 구현해야 합니다.

SideEffect를 사용하는 경우 로딩 상태도 추가하는 것이 좋습니다:

sealed interface EmploymentDetailSideEffect { data object BadRequest : EmploymentDetailSideEffect data object Success : EmploymentDetailSideEffect + data object Loading : EmploymentDetailSideEffect }

그리고 fetchEmploymentStatus() 함수 시작 시 Loading 상태를 발행하도록 수정:

internal fun fetchEmploymentStatus() { + postSideEffect(EmploymentDetailSideEffect.Loading) viewModelScope.launch(Dispatchers.IO) { fetchEmploymentStatusUseCase().onSuccess { // ... postSideEffect(EmploymentDetailSideEffect.Success) }.onFailure { postSideEffect(EmploymentDetailSideEffect.BadRequest) } } }

📝 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

sealed interface EmploymentDetailSideEffect {

data object BadRequest : EmploymentDetailSideEffect

data object Success : EmploymentDetailSideEffect

}

sealed interface EmploymentDetailSideEffect {

data object BadRequest : EmploymentDetailSideEffect

data object Success : EmploymentDetailSideEffect

data object Loading : EmploymentDetailSideEffect

}

Suggested change

sealed interface EmploymentDetailSideEffect {

data object BadRequest : EmploymentDetailSideEffect

data object Success : EmploymentDetailSideEffect

}

internal fun fetchEmploymentStatus() {

postSideEffect(EmploymentDetailSideEffect.Loading)

viewModelScope.launch(Dispatchers.IO) {

fetchEmploymentStatusUseCase().onSuccess {

// ...

postSideEffect(EmploymentDetailSideEffect.Success)

}.onFailure {

postSideEffect(EmploymentDetailSideEffect.BadRequest)

}

}

}

coderabbitai · 2025-03-05T11:13:37Z

feature/employment/src/main/java/team/retum/employment/viewmodel/EmploymentViewModel.kt

+    internal fun fetchEmploymentCount() {
+        viewModelScope.launch(Dispatchers.IO) {
+            fetchEmploymentCountUseCase().onSuccess {
+                setState {
+                    val rate = if (it.passCount == 0L || it.totalStudentCount == 0L) {
+                        0f
+                    } else {
+                        it.passCount.toFloat() / it.totalStudentCount.toFloat() * 100f
+                    }
+                    state.value.copy(
+                        rate = DecimalFormat("##.#").format(rate),
+                        totalStudentCount = it.totalStudentCount,
+                        passCount = it.passCount,
+                    )
+                }
+            }
+        }
+    }


🛠️ Refactor suggestion

에러 처리 로직 추가 필요

fetchEmploymentCount 메서드에서 API 호출의 성공 케이스만 처리하고 있으며, 실패 케이스에 대한 처리가 없습니다. 에러가 발생할 경우 UI에 적절히 상태를 업데이트하고 사이드 이펙트를 발생시켜야 합니다.

fetchEmploymentCountUseCase().onSuccess { setState { val rate = if (it.passCount == 0L || it.totalStudentCount == 0L) { 0f } else { it.passCount.toFloat() / it.totalStudentCount.toFloat() * 100f } state.value.copy( rate = DecimalFormat("##.#").format(rate), totalStudentCount = it.totalStudentCount, passCount = it.passCount, ) } + }.onFailure { + postSideEffect(EmploymentSideEffect.BadRequest) }

📝 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

internal fun fetchEmploymentCount() {

viewModelScope.launch(Dispatchers.IO) {

fetchEmploymentCountUseCase().onSuccess {

setState {

val rate = if (it.passCount == 0L || it.totalStudentCount == 0L) {

0f

} else {

it.passCount.toFloat() / it.totalStudentCount.toFloat() * 100f

}

state.value.copy(

rate = DecimalFormat("##.#").format(rate),

totalStudentCount = it.totalStudentCount,

passCount = it.passCount,

)

}

}

}

}

internal fun fetchEmploymentCount() {

viewModelScope.launch(Dispatchers.IO) {

fetchEmploymentCountUseCase().onSuccess {

setState {

val rate = if (it.passCount == 0L || it.totalStudentCount == 0L) {

0f

} else {

it.passCount.toFloat() / it.totalStudentCount.toFloat() * 100f

}

state.value.copy(

rate = DecimalFormat("##.#").format(rate),

totalStudentCount = it.totalStudentCount,

passCount = it.passCount,

)

}

}.onFailure {

postSideEffect(EmploymentSideEffect.BadRequest)

}

}

}

coderabbitai · 2025-03-05T11:13:37Z

feature/employment/src/main/java/team/retum/employment/ui/EmploymentScreen.kt

+                            text = "전체 통계",
+                            style = JobisTypography.Body,


🛠️ Refactor suggestion

하드코딩된 문자열을 문자열 리소스로 대체 필요

"전체 통계"라는 문자열이 하드코딩되어 있습니다. 다국어 지원과 일관성을 위해 이를 문자열 리소스로 추출하는 것이 좋습니다.

- text = "전체 통계", + text = stringResource(id = R.string.total_statistics),

R.string.total_statistics에 해당하는 문자열 리소스를 추가해야 합니다.

📝 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

text = "전체 통계",

style = JobisTypography.Body,

text = stringResource(id = R.string.total_statistics),

style = JobisTypography.Body,

coderabbitai · 2025-03-05T11:13:37Z

feature/employment/src/main/java/team/retum/employment/ui/EmploymentScreen.kt

+            Row(
+                modifier = Modifier.fillMaxWidth(),
+                horizontalArrangement = Arrangement.SpaceBetween
+            ) {
+                ClassEmploymentButton(
+                    onClassClick = onClassClick,
+                    image = team.retum.design_system.R.drawable.ic_computer,
+                    text = "1반",
+                )
+                ClassEmploymentButton(
+                    onClassClick = onClassClick,
+                    image = team.retum.design_system.R.drawable.ic_computer,
+                    text = "2반",
+                )
+            }
+            Row(
+                modifier = Modifier.fillMaxWidth(),
+                horizontalArrangement = Arrangement.SpaceBetween
+            ) {
+                ClassEmploymentButton(
+                    onClassClick = onClassClick,
+                    image = team.retum.design_system.R.drawable.ic_spanner,
+                    text = "3반",
+                )
+                ClassEmploymentButton(
+                    onClassClick = onClassClick,
+                    image = team.retum.design_system.R.drawable.ic_robot,
+                    text = "4반",
+                )
+            }


🛠️ Refactor suggestion

클래스 버튼 로직 개선 필요

현재 각 반(1반, 2반, 3반, 4반)에 대한 버튼이 하드코딩되어 있습니다. 이는 유지 관리가 어렵고 확장성이 떨어집니다. 리스트와 forEach 또는 forEachIndexed를 사용하여 이 코드를 간결하게 작성할 수 있습니다.

- Row( - modifier = Modifier.fillMaxWidth(), - horizontalArrangement = Arrangement.SpaceBetween - ) { - ClassEmploymentButton( - onClassClick = onClassClick, - image = team.retum.design_system.R.drawable.ic_computer, - text = "1반", - ) - ClassEmploymentButton( - onClassClick = onClassClick, - image = team.retum.design_system.R.drawable.ic_computer, - text = "2반", - ) - } - Row( - modifier = Modifier.fillMaxWidth(), - horizontalArrangement = Arrangement.SpaceBetween - ) { - ClassEmploymentButton( - onClassClick = onClassClick, - image = team.retum.design_system.R.drawable.ic_spanner, - text = "3반", - ) - ClassEmploymentButton( - onClassClick = onClassClick, - image = team.retum.design_system.R.drawable.ic_robot, - text = "4반", - ) - } + val classImages = listOf( + team.retum.design_system.R.drawable.ic_computer, + team.retum.design_system.R.drawable.ic_computer, + team.retum.design_system.R.drawable.ic_spanner, + team.retum.design_system.R.drawable.ic_robot + ) + + for (rowIndex in 0..1) { + Row( + modifier = Modifier.fillMaxWidth(), + horizontalArrangement = Arrangement.SpaceBetween + ) { + for (colIndex in 0..1) { + val classIndex = rowIndex * 2 + colIndex + ClassEmploymentButton( + onClassClick = onClassClick, + image = classImages[classIndex], + text = "${classIndex + 1}반", + ) + } + } + }

📝 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

Row(

modifier = Modifier.fillMaxWidth(),

horizontalArrangement = Arrangement.SpaceBetween

) {

ClassEmploymentButton(

onClassClick = onClassClick,

image = team.retum.design_system.R.drawable.ic_computer,

text = "1반",

)

ClassEmploymentButton(

onClassClick = onClassClick,

image = team.retum.design_system.R.drawable.ic_computer,

text = "2반",

)

}

Row(

modifier = Modifier.fillMaxWidth(),

horizontalArrangement = Arrangement.SpaceBetween

) {

ClassEmploymentButton(

onClassClick = onClassClick,

image = team.retum.design_system.R.drawable.ic_spanner,

text = "3반",

)

ClassEmploymentButton(

onClassClick = onClassClick,

image = team.retum.design_system.R.drawable.ic_robot,

text = "4반",

)

}

val classImages = listOf(

team.retum.design_system.R.drawable.ic_computer,

team.retum.design_system.R.drawable.ic_computer,

team.retum.design_system.R.drawable.ic_spanner,

team.retum.design_system.R.drawable.ic_robot

)

for (rowIndex in 0..1) {

Row(

modifier = Modifier.fillMaxWidth(),

horizontalArrangement = Arrangement.SpaceBetween

) {

for (colIndex in 0..1) {

val classIndex = rowIndex * 2 + colIndex

ClassEmploymentButton(

onClassClick = onClassClick,

image = classImages[classIndex],

text = "${classIndex + 1}반",

)

}

}

}

coderabbitai

Actionable comments posted: 0

🧹 Nitpick comments (3)

feature/employment/src/main/java/team/retum/employment/ui/EmploymentScreen.kt (3)

60-60: 아직 구현되지 않은 네비게이션 로직
Employment 함수 내부에 // TODO :: 네비게이션 추가 및 부수라고만 남겨두고 실제 구현은 비어 있습니다. 후속 작업에서 해당 로직을 구현하거나 연동하는 과정을 진행해야 할 것으로 보입니다.

필요하시다면 네비게이션 로직 구현 예시나 추가 설정에 관한 안내를 드릴 수 있으니 말씀해 주세요.

62-71: 애니메이션 값이 하위 컴포저블에 반영되지 않음
부모 컴포저블(Employment)에서 Animatable을 통해 애니메이션을 진행하지만, 실제로 EmploymentScreen에 넘기는 인자는 단순히 rate 값만 전달하고 있습니다. 이로 인해 animatedValue.value가 UI에 반영되지 않아 애니메이션 효과가 적용되지 않습니다.

EmploymentScreen으로 animatedValue.value를 넘기거나, CircleProgress 내부 로직에서 직접 애니메이션을 처리하도록 변경해 주시면 의도한 애니메이션 효과를 구현할 수 있습니다.

261-308: CircleProgress 내부 애니메이션 로직이 누락됨
CircleProgress에서 Animatable(percentage)로 초기화만 진행하고, 실제로 값을 변화시키는 animateTo 등이 호출되지 않습니다. 이 상태로는 원형 그래프가 애니메이션 없이 고정된 상태로 표시됩니다.

LaunchedEffect(percentage)나 다른 트리거를 사용해 실제로 animateTo(percentage)를 호출하거나, 상위에서 이미 애니메이션을 처리해 전달하는 구조로 개선해주세요.

📜 Review details

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

📥 Commits

Reviewing files that changed from the base of the PR and between 597e908 and 560e029.

📒 Files selected for processing (4)

core/common/src/main/java/team/retum/common/utils/ResourceKeys.kt (1 hunks)
feature/employment/build.gradle.kts (1 hunks)
feature/employment/src/main/java/team/retum/employment/ui/EmploymentScreen.kt (1 hunks)
feature/employment/src/main/java/team/retum/employment/viewmodel/EmploymentViewModel.kt (1 hunks)

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

feature/employment/src/main/java/team/retum/employment/viewmodel/EmploymentViewModel.kt
core/common/src/main/java/team/retum/common/utils/ResourceKeys.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 (6)

feature/employment/build.gradle.kts (4)

1-7: 플러그인 설정 확인
플러그인 블록에서 Android 라이브러리, Kotlin Android, Hilt, KSP, 그리고 ktlint 플러그인이 명확하게 적용되어 있어 모듈의 요구사항을 잘 반영하고 있습니다.

9-10: 커스텀 플러그인 적용 적절
apply<CommonGradlePlugin>()와 apply<ComposeGradlePlugin>()를 통해 공통 설정과 Compose 관련 구성 요소를 모듈에 손쉽게 적용한 점이 좋습니다.

12-18: Android 구성 설정 검토
android 블록 내에서 namespace를 명시하고, kotlinOptions를 통해 JVM 타겟을 ProjectProperties.JVM_TARGET으로 설정한 부분이 모듈의 일관성을 유지하는 데 도움이 됩니다.

20-23: 의존성 추가 검토
implementation(libs.androidx.ui.tooling.preview.android) 의존성을 추가하여 AndroidX UI 툴링 미리보기 기능을 활성화한 점이 적절합니다. 향후 툴링 관련 버전 업데이트나 충돌 가능성을 주기적으로 점검하면 좋겠습니다.

feature/employment/src/main/java/team/retum/employment/ui/EmploymentScreen.kt (2)

185-187: 하드코딩된 문자열을 문자열 리소스로 대체 필요
과거 리뷰에서도 지적된 내용과 동일합니다. 다국어 지원 및 유지보수를 위해 하드코딩된 "전체 통계" 문자열을 리소스로 추출하는 것을 권장드립니다.

-                            text = "전체 통계",
+                            text = stringResource(id = R.string.total_statistics),

218-247: 클래스 버튼 로직 개선 필요
이전 리뷰에서 지적된 것처럼 “1반, 2반, 3반, 4반”과 관련된 버튼을 반복 작성하고 있습니다. 추후 클래스 수가 늘어나거나 변경될 경우 확장성이 떨어지므로, List와 forEachIndexed 등을 활용해 동적으로 생성하는 방식을 권장합니다.

-            Row(
-                modifier = Modifier.fillMaxWidth(),
-                horizontalArrangement = Arrangement.SpaceBetween
-            ) {
-                ClassEmploymentButton(
-                    onClassClick = onClassClick,
-                    image = team.retum.design_system.R.drawable.ic_computer,
-                    text = "1반",
-                )
-                ClassEmploymentButton(
-                    onClassClick = onClassClick,
-                    image = team.retum.design_system.R.drawable.ic_computer,
-                    text = "2반",
-                )
-            }
-            Row(
-                modifier = Modifier.fillMaxWidth(),
-                horizontalArrangement = Arrangement.SpaceBetween
-            ) {
-                ClassEmploymentButton(
-                    onClassClick = onClassClick,
-                    image = team.retum.design_system.R.drawable.ic_spanner,
-                    text = "3반",
-                )
-                ClassEmploymentButton(
-                    onClassClick = onClassClick,
-                    image = team.retum.design_system.R.drawable.ic_robot,
-                    text = "4반",
-                )
-            }
+            val classImages = listOf(
+                team.retum.design_system.R.drawable.ic_computer,
+                team.retum.design_system.R.drawable.ic_computer,
+                team.retum.design_system.R.drawable.ic_spanner,
+                team.retum.design_system.R.drawable.ic_robot
+            )
+
+            for (rowIndex in 0..1) {
+                Row(
+                    modifier = Modifier.fillMaxWidth(),
+                    horizontalArrangement = Arrangement.SpaceBetween
+                ) {
+                    for (colIndex in 0..1) {
+                        val classIndex = rowIndex * 2 + colIndex
+                        ClassEmploymentButton(
+                            onClassClick = onClassClick,
+                            image = classImages[classIndex],
+                            text = "${classIndex + 1}반",
+                        )
+                    }
+                }
+            }

coderabbitai

Actionable comments posted: 2

♻️ Duplicate comments (4)

feature/employment/src/main/java/team/retum/employment/ui/EmploymentDetailScreen.kt (3)
14-21: ⚠️ Potential issue

ViewModel 데이터를 활용하지 않고 있습니다.

ViewModel을 초기화하지만 실제로 데이터를 가져오거나 상태를 관찰하고 있지 않습니다. 대신 빈 리스트를 EmploymentDetailScreen에 전달하고 있습니다.

다음과 같이 ViewModel 상태를 관찰하도록 수정하세요:
@Composable
fun EmploymentDetail(
    employmentDetailViewModel: EmploymentDetailViewModel = hiltViewModel(),
) {
+    val state by employmentDetailViewModel.state.collectAsState()
+    
+    LaunchedEffect(Unit) {
+        employmentDetailViewModel.fetchEmploymentStatus()
+    }
+    
    EmploymentDetailScreen(
-        companyItem = emptyList(),
+        companyItem = state.companyItems,
    )
}
27-42: ⚠️ Potential issue

파라미터로 전달된 데이터를 사용하지 않고 있습니다.

companyItem 파라미터를 전달받았지만, 이를 사용하지 않고 하드코딩된 빈 항목 리스트를 생성하여 사용하고 있습니다.

다음과 같이 전달받은 파라미터를 사용하도록 수정하세요:
Column(
    modifier = Modifier
        .fillMaxSize(),
) {
-    val arr: List<CompanyItem> = listOf(
-        CompanyItem(
-            companyName = "",
-            logoUrl = "",
-        ),
-    )
    LazyHorizontalGrid(
+        modifier = Modifier.fillMaxWidth(),
        rows = GridCells.Fixed(4),
    ) {
-        items(items = arr) { person ->
+        items(items = companyItem) { company ->
+            CompanyIcon(
+                companyName = company.companyName,
+                logoUrl = company.logoUrl
+            )
        }
    }
}
40-41: ⚠️ Potential issue

items 블록이 비어 있습니다.

LazyHorizontalGrid의 items 블록 내부가 비어있어 아무 내용도 표시되지 않습니다.

다음과 같이 CompanyIcon을 호출하는 코드를 추가하세요:
items(items = arr) { person ->
+    CompanyIcon(
+        companyName = person.companyName,
+        logoUrl = person.logoUrl
+    )
}
참고: CompanyIcon 함수를 구현해야 합니다.
feature/employment/src/main/java/team/retum/employment/ui/EmploymentScreen.kt (1)
186-187: 🛠️ Refactor suggestion

하드코딩된 문자열을 문자열 리소스로 대체 필요

"전체 통계"라는 문자열이 하드코딩되어 있습니다. 다국어 지원과 일관성을 위해 이를 문자열 리소스로 추출하는 것이 좋습니다.
-                            text = "전체 통계",
+                            text = stringResource(id = R.string.total_statistics),
R.string.total_statistics에 해당하는 문자열 리소스를 추가해야 합니다.

🧹 Nitpick comments (3)

feature/home/src/main/java/team/retum/home/ui/HomeScreen.kt (1)
323-331: 고용 현황 상세 페이지 이동 버튼이 잘 구현되었습니다.

기존 텍스트 표시 방식에서 버튼으로 변경하여 사용자 경험이 개선되었습니다. 버튼 텍스트 "현황 보러 가기 ->"가 목적을 명확히 표현하고 있습니다.

한 가지 제안할 점은 버튼 텍스트를 국제화(i18n)하는 것입니다. 현재 하드코딩된 문자열을 strings.xml 리소스로 이동시키면 다국어 지원이 용이해집니다.

텍스트 문자열을 리소스로 이동하는 것이 좋습니다:
-                text = "현황 보러 가기 ->",
+                text = stringResource(id = R.string.view_employment_status),
strings.xml에 다음을 추가하세요:
<string name="view_employment_status">현황 보러 가기 -></string>
feature/employment/src/main/java/team/retum/employment/ui/EmploymentScreen.kt (2)
66-68: ViewModel 함수 호출 방식 개선 필요

with(employmentViewModel) 블록을 사용하는 대신 직접 호출하는 것이 더 간결합니다.

다음과 같이 리팩토링하세요:
LaunchedEffect(Unit) {
    animatedValue.animateTo(
        targetValue = rate,
        animationSpec = tween(durationMillis = 1000, easing = LinearEasing),
    )
-    with(employmentViewModel) {
-        fetchEmploymentCount()
-    }
+    employmentViewModel.fetchEmploymentCount()
}
318-319: Surface의 modifier와 onClick 매개변수 개선 필요

Surface의 modifier에 padding이 비어 있고, onClick 함수가 직접 전달되고 있습니다.

다음과 같이 개선하세요:
Surface(
-    modifier = Modifier.padding(),
+    modifier = Modifier,
-    onClick = onClassClick,
+    onClick = { onClassClick() },
    color = JobisTheme.colors.inverseSurface,
    shape = RoundedCornerShape(8.dp),
)

📜 Review details

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

📥 Commits

Reviewing files that changed from the base of the PR and between 560e029 and 412a4c2.

📒 Files selected for processing (8)

core/domain/src/main/java/team/retum/usecase/entity/application/EmploymentStatusEntity.kt (1 hunks)
core/network/src/main/java/team/retum/network/model/response/application/FetchEmploymentStatusResponse.kt (1 hunks)
feature/employment/build.gradle.kts (1 hunks)
feature/employment/src/main/java/team/retum/employment/navigation/EmploymentNavigation.kt (1 hunks)
feature/employment/src/main/java/team/retum/employment/ui/EmploymentDetailScreen.kt (1 hunks)
feature/employment/src/main/java/team/retum/employment/ui/EmploymentScreen.kt (1 hunks)
feature/employment/src/main/java/team/retum/employment/viewmodel/EmploymentDetailViewModel.kt (1 hunks)
feature/home/src/main/java/team/retum/home/ui/HomeScreen.kt (11 hunks)

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

feature/employment/src/main/java/team/retum/employment/navigation/EmploymentNavigation.kt
feature/employment/build.gradle.kts
core/domain/src/main/java/team/retum/usecase/entity/application/EmploymentStatusEntity.kt
core/network/src/main/java/team/retum/network/model/response/application/FetchEmploymentStatusResponse.kt
feature/employment/src/main/java/team/retum/employment/viewmodel/EmploymentDetailViewModel.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 (5)

feature/home/src/main/java/team/retum/home/ui/HomeScreen.kt (2)

23-23: 필요한 import 문이 적절하게 추가되었습니다.

새로운 버튼 컴포넌트와 관련 스타일링을 위한 import가 잘 추가되었습니다.

Also applies to: 39-39, 61-61, 63-63

83-83: 컴포넌트 계층 구조를 통한 이벤트 전달이 잘 구현되었습니다.

상위 컴포넌트에서 하위 컴포넌트로 onEmploymentClick 콜백을 전달하는 패턴이 일관되게 적용되었습니다. 이는 Compose의 단방향 데이터 흐름 패턴을 잘 따르고 있으며, 사용자가 고용 현황 상세 페이지로 이동할 수 있는 기능을 효과적으로 제공합니다.

Also applies to: 130-130, 154-154, 186-186, 228-228, 270-270, 309-309

feature/employment/src/main/java/team/retum/employment/ui/EmploymentScreen.kt (3)

58-58: 완료되지 않은 TODO 주석이 있습니다.

네비게이션 관련 TODO 주석이 완료되지 않았습니다.

네비게이션 관련 작업이 완료되었는지 확인하고, 필요한 경우 TODO 주석을 제거하세요.

87-87: 완료되지 않은 TODO 주석이 있습니다.

UI 퍼블리싱 관련 TODO 주석이 완료되지 않았습니다.

UI 퍼블리싱 작업이 완료되었는지 확인하고, 필요한 경우 TODO 주석을 제거하세요.

218-247: 클래스 버튼 로직 개선 필요

현재 각 반(1반, 2반, 3반, 4반)에 대한 버튼이 하드코딩되어 있습니다. 이는 유지 관리가 어렵고 확장성이 떨어집니다.

리스트와 forEach 또는 forEachIndexed를 사용하여 이 코드를 간결하게 작성할 수 있습니다:

-            Row(
-                modifier = Modifier.fillMaxWidth(),
-                horizontalArrangement = Arrangement.SpaceBetween,
-            ) {
-                ClassEmploymentButton(
-                    onClassClick = onClassClick,
-                    image = team.retum.design_system.R.drawable.ic_computer,
-                    text = "1반",
-                )
-                ClassEmploymentButton(
-                    onClassClick = onClassClick,
-                    image = team.retum.design_system.R.drawable.ic_computer,
-                    text = "2반",
-                )
-            }
-            Row(
-                modifier = Modifier.fillMaxWidth(),
-                horizontalArrangement = Arrangement.SpaceBetween,
-            ) {
-                ClassEmploymentButton(
-                    onClassClick = onClassClick,
-                    image = team.retum.design_system.R.drawable.ic_spanner,
-                    text = "3반",
-                )
-                ClassEmploymentButton(
-                    onClassClick = onClassClick,
-                    image = team.retum.design_system.R.drawable.ic_robot,
-                    text = "4반",
-                )
-            }
+            val classImages = listOf(
+                team.retum.design_system.R.drawable.ic_computer,
+                team.retum.design_system.R.drawable.ic_computer,
+                team.retum.design_system.R.drawable.ic_spanner,
+                team.retum.design_system.R.drawable.ic_robot
+            )
+            
+            for (rowIndex in 0..1) {
+                Row(
+                    modifier = Modifier.fillMaxWidth(),
+                    horizontalArrangement = Arrangement.SpaceBetween
+                ) {
+                    for (colIndex in 0..1) {
+                        val classIndex = rowIndex * 2 + colIndex
+                        ClassEmploymentButton(
+                            onClassClick = onClassClick,
+                            image = classImages[classIndex],
+                            text = "${classIndex + 1}반",
+                        )
+                    }
+                }
+            }

coderabbitai · 2025-03-06T02:19:34Z

feature/employment/src/main/java/team/retum/employment/ui/EmploymentDetailScreen.kt

+            }
+        }
+    }
+}


⚠️ Potential issue

CompanyIcon 함수가 누락되었습니다.

코드에서 CompanyIcon 함수를 사용하고 있지만, 함수 구현이 없습니다.

다음과 같이 CompanyIcon 함수를 구현하세요:

@Composable fun CompanyIcon( companyName: String, logoUrl: String ) { Column( modifier = Modifier .padding(8.dp) .fillMaxWidth(), horizontalAlignment = Alignment.CenterHorizontally ) { AsyncImage( model = logoUrl, contentDescription = "$companyName 로고", modifier = Modifier .size(64.dp) .clip(CircleShape), contentScale = ContentScale.Crop, placeholder = painterResource(id = R.drawable.ic_company_placeholder) ) Spacer(modifier = Modifier.height(4.dp)) Text( text = companyName, style = MaterialTheme.typography.caption, maxLines = 1, overflow = TextOverflow.Ellipsis ) } }

AsyncImage 사용을 위해 다음 import 문도 추가해야 합니다:

import coil.compose.AsyncImage import coil.request.ImageRequest import androidx.compose.foundation.shape.CircleShape import androidx.compose.material3.MaterialTheme import androidx.compose.material3.Text import androidx.compose.ui.draw.clip import androidx.compose.ui.layout.ContentScale import androidx.compose.ui.res.painterResource import androidx.compose.ui.text.style.TextOverflow

coderabbitai · 2025-03-06T02:19:34Z

feature/employment/src/main/java/team/retum/employment/ui/EmploymentScreen.kt

+    primaryColor: Color = JobisTheme.colors.secondaryContainer,
+    secondaryColor: Color = JobisTheme.colors.onPrimary,
+) {
+    val animatedValue = remember { Animatable(percentage) }


⚠️ Potential issue

CircleProgress에서 애니메이션 값이 재사용되지 않음

CircleProgress 컴포저블에서 percentage 파라미터가 변경될 때 애니메이션이 시작되지 않습니다.

다음과 같이 LaunchedEffect를 추가하여 percentage 값이 변경될 때마다 애니메이션이 시작되도록 수정하세요:

fun CircleProgress( percentage: Float, modifier: Modifier = Modifier, radius: Dp = 120.dp, strokeWidth: Dp = 25.dp, primaryColor: Color = JobisTheme.colors.secondaryContainer, secondaryColor: Color = JobisTheme.colors.onPrimary, ) { val animatedValue = remember { Animatable(percentage) } + + LaunchedEffect(percentage) { + animatedValue.animateTo( + targetValue = percentage, + animationSpec = tween(durationMillis = 1000, easing = LinearEasing) + ) + }

📝 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

val animatedValue = remember { Animatable(percentage) }

fun CircleProgress(

percentage: Float,

modifier: Modifier = Modifier,

radius: Dp = 120.dp,

strokeWidth: Dp = 25.dp,

primaryColor: Color = JobisTheme.colors.secondaryContainer,

secondaryColor: Color = JobisTheme.colors.onPrimary,

) {

val animatedValue = remember { Animatable(percentage) }

LaunchedEffect(percentage) {

animatedValue.animateTo(

targetValue = percentage,

animationSpec = tween(durationMillis = 1000, easing = LinearEasing)

)

}

// ... rest of the composable code

}

parkuiery · 2025-03-06T02:34:33Z

app/build.gradle.kts

+//            splits.abi.isEnable = false
+//            aaptOptions.cruncherEnabled = false
+//            splits.density.isEnable = false
+//            aaptOptions.cruncherEnabled = false


주석은 제거 해주세요

parkuiery · 2025-03-06T03:06:53Z

core/design-system/src/main/res/drawable/ic_computer.png

xml 형식으로 리소스가 깨지지 않는다면 png 사용은 지양해주세요.
png를 사용하면 파일의 크기가 커져 앱 크기에 영향을 줄 수 있어요.

parkuiery · 2025-03-06T03:10:00Z

core/domain/src/main/java/team/retum/usecase/entity/application/EmploymentStatusEntity.kt

+}
+
+internal fun FetchEmploymentStatusResponse.toEntity() = EmploymentStatusEntity(
+    classes = this.classes.map { it.toEntity() }, // TODO :: 이 구조 이해 필요


todo 주석은 추후에 리펙토링이나 변경될 사항이 생길 경우에만 달아주세용

parkuiery · 2025-03-06T03:10:24Z

core/domain/src/main/java/team/retum/usecase/entity/application/EmploymentStatusEntity.kt

+import team.retum.network.model.response.application.FetchEmploymentStatusResponse
+
+data class EmploymentStatusEntity(
+    val classes: List<ClassEmploymentStatusEntity>,


네이밍을 classes로 하신 이유가 있을까요?

parkuiery · 2025-03-06T03:16:11Z

.../domain/src/main/java/team/retum/usecase/usecase/application/FetchEmploymentStatusUseCase.kt

+    private val applicationRepository: ApplicationRepository,
+) {
+    suspend operator fun invoke() = runCatching {
+        applicationRepository.fetchEmploymentStatus().toEntity()


이 부분은 전체 리펙토링을 해야 하는 부분인데 도메인 레이어에서는 비스니스 로직과 캡슐화의 역할을 가지고 있어 맵핑 작업은 data 레이어에서 할 수 있도록 변경할 예정입니다. 참고해주세요.

parkuiery · 2025-03-06T05:24:56Z

feature/home/src/main/java/team/retum/home/ui/HomeScreen.kt

+                modifier = Modifier
+                    .wrapContentSize()
+                    .background(Color.Transparent)
+                    .fillMaxWidth(0.4f),


0.4f는 왜 사용하신건가요?

parkuiery · 2025-03-06T05:25:36Z

gradle/libs.versions.toml

@@ -17,7 +17,7 @@ firebase-messaging = "23.4.1"

 hilt-android-gradle-plugin = "2.51.1"
 junit = "4.13.2"
-android-gradle = "8.2.2"
+android-gradle = "8.7.0"


Gradle 버전 업그레이드 하신 이유가 무엇인가요?

parkuiery · 2025-03-06T05:26:24Z

gradle/libs.versions.toml

+androidx-tiles-tooling-preview = { group = "androidx.wear.tiles", name = "tiles-tooling-preview", version.ref = "tiles-tooling-preview" }
+androidx-ui-tooling-preview-android = { group = "androidx.compose.ui", name = "ui-tooling-preview-android", version.ref = "ui-tooling-preview-android" }


preview는 기본적으로 제공되는데 추가하신 이유가 무엇인가요?

parkuiery · 2025-03-06T05:27:01Z

gradle/wrapper/gradle-wrapper.properties

-distributionUrl=https\://services.gradle.org/distributions/gradle-8.2-bin.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-8.9-bin.zip
+networkTimeout=10000
+validateDistributionUrl=true


이걸 추가하면 어떤것들이 변경이 되나요?

parkuiery · 2025-03-06T05:27:42Z

gradlew

이 파일을 수정하신 이유가 무엇이고 어떤것들이 변경되는건가요?

uson1004 added 18 commits February 12, 2025 17:39

chore :: employment 모듈 생성

04668ae

feat :: employment 기본 파일 추가

e5f0b9c

feat :: 모듈 내 파일 기준 틀 세팅

3c49224

feat :: employment API 모델 생성

d231f85

feat :: Application Repository 기능 추가

b0d1567

feat :: EmploymentStatusEntity 생성

cbfe6db

chore :: @Preview gradle 의존성 추가

92fd187

refactor :: banner UI 상태 속성 변경

71c608a

feat :: employment navigator 외부 설정

2b3a40e

build :: 컴파일 자바 버전 변경

bf463c6

ksp 버전 호환

feat :: employment 그래프 구현

850397a

feat :: 이미지 추가

a6b8f7d

refactor :: debug split api enable

23cfd1a

feat :: 리소스 추가

aa7f762

feat :: ui 상태 올림 및 네비게이션 등록

ee766b0

feat :: employmentDetail Screen 추가

2ef2117

feat :: Detailscreen companyItem 모델 추가

60ec96a

feat :: DetailScreen viewmodel 상태 정의

597e908

uson1004 added 기능 구현 비즈니스 로직을 작성합니다. 퍼블리싱 화면을 퍼블리싱 합니다. labels Mar 5, 2025

uson1004 requested review from Tmdhoon2, rladmsdh and parkuiery March 5, 2025 11:08

uson1004 self-assigned this Mar 5, 2025

uson1004 requested a review from a team as a code owner March 5, 2025 11:08

uson1004 linked an issue Mar 5, 2025 that may be closed by this pull request

취업률 상세 조회 페이지 제작 #448

Open

2 tasks

uson1004 closed this Mar 5, 2025

coderabbitai bot reviewed Mar 5, 2025

View reviewed changes

chore :: 코드 정렬

4117fb2

uson1004 reopened this Mar 5, 2025

chore :: unuse import 제거

560e029

coderabbitai bot reviewed Mar 5, 2025

View reviewed changes

uson1004 changed the title ~~Feature/448 취업률 상세 조회 페이지 제작~~ :: (#448) 취업률 상세 조회 페이지 제작 Mar 5, 2025

uson1004 changed the title ~~:: (#448) 취업률 상세 조회 페이지 제작~~ 🔗 :: (#448) 취업률 상세 조회 페이지 제작 Mar 5, 2025

uson1004 added 2 commits March 6, 2025 10:23

build :: preview 의존성 제거

49c993f

chore :: ktlint 코드 컨밴션 수정

412a4c2

coderabbitai bot reviewed Mar 6, 2025

View reviewed changes

parkuiery requested changes Mar 6, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🔗 :: (#448) 취업률 상세 조회 페이지 제작 #449

🔗 :: (#448) 취업률 상세 조회 페이지 제작 #449

uson1004 commented Mar 5, 2025 •

edited

Loading

coderabbitai bot commented Mar 5, 2025 •

edited

Loading

Chat

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (`.coderabbit.yaml`)

Documentation and Community

coderabbitai bot left a comment

coderabbitai bot Mar 5, 2025

coderabbitai bot Mar 5, 2025

coderabbitai bot Mar 5, 2025

coderabbitai bot Mar 5, 2025

coderabbitai bot Mar 5, 2025

coderabbitai bot Mar 5, 2025

coderabbitai bot Mar 5, 2025

coderabbitai bot Mar 5, 2025

coderabbitai bot Mar 5, 2025

coderabbitai bot Mar 5, 2025

coderabbitai bot left a comment

coderabbitai bot left a comment

coderabbitai bot Mar 6, 2025

coderabbitai bot Mar 6, 2025

parkuiery Mar 6, 2025

parkuiery Mar 6, 2025

parkuiery Mar 6, 2025

parkuiery Mar 6, 2025

parkuiery Mar 6, 2025

parkuiery Mar 6, 2025

parkuiery Mar 6, 2025

parkuiery Mar 6, 2025

parkuiery Mar 6, 2025

parkuiery Mar 6, 2025

		@@ -1,5 +1,7 @@
		package team.retum.common.utils

		import android.provider.Telephony.Mms.Rate

		route = "$NAVIGATION_EMPLOYMENT/${ResourceKeys}",
		arguments = listOf(

-sealed interface EmploymentDetailSideEffect {
-    data object BadRequest : EmploymentDetailSideEffect
-    data object Success : EmploymentDetailSideEffect
-}
+internal fun fetchEmploymentStatus() {
+    postSideEffect(EmploymentDetailSideEffect.Loading)
+    viewModelScope.launch(Dispatchers.IO) {
+        fetchEmploymentStatusUseCase().onSuccess {
+            // ...
+            postSideEffect(EmploymentDetailSideEffect.Success)
+        }.onFailure {
+            postSideEffect(EmploymentDetailSideEffect.BadRequest)
+        }
+    }
+}

-    val animatedValue = remember { Animatable(percentage) }
+fun CircleProgress(
+    percentage: Float,
+    modifier: Modifier = Modifier,
+    radius: Dp = 120.dp,
+    strokeWidth: Dp = 25.dp,
+    primaryColor: Color = JobisTheme.colors.secondaryContainer,
+    secondaryColor: Color = JobisTheme.colors.onPrimary,
+) {
+    val animatedValue = remember { Animatable(percentage) }
+    LaunchedEffect(percentage) {
+        animatedValue.animateTo(
+            targetValue = percentage,
+            animationSpec = tween(durationMillis = 1000, easing = LinearEasing)
+        )
+    }
+    // ... rest of the composable code
+}

		androidx-tiles-tooling-preview = { group = "androidx.wear.tiles", name = "tiles-tooling-preview", version.ref = "tiles-tooling-preview" }
		androidx-ui-tooling-preview-android = { group = "androidx.compose.ui", name = "ui-tooling-preview-android", version.ref = "ui-tooling-preview-android" }

🔗 :: (#448) 취업률 상세 조회 페이지 제작 #449

Are you sure you want to change the base?

🔗 :: (#448) 취업률 상세 조회 페이지 제작 #449

Conversation

uson1004 commented Mar 5, 2025 • edited Loading

개요

작업 내용

할 말

Summary by CodeRabbit

coderabbitai bot commented Mar 5, 2025 • edited Loading

Walkthrough

Changes

Sequence Diagram(s)

Suggested reviewers

Poem

Chat

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (.coderabbit.yaml)

Documentation and Community

coderabbitai bot left a comment

Choose a reason for hiding this comment

coderabbitai bot Mar 5, 2025

Choose a reason for hiding this comment

coderabbitai bot Mar 5, 2025

Choose a reason for hiding this comment

coderabbitai bot Mar 5, 2025

Choose a reason for hiding this comment

coderabbitai bot Mar 5, 2025

Choose a reason for hiding this comment

coderabbitai bot Mar 5, 2025

Choose a reason for hiding this comment

coderabbitai bot Mar 5, 2025

Choose a reason for hiding this comment

coderabbitai bot Mar 5, 2025

Choose a reason for hiding this comment

coderabbitai bot Mar 5, 2025

Choose a reason for hiding this comment

coderabbitai bot Mar 5, 2025

Choose a reason for hiding this comment

coderabbitai bot Mar 5, 2025

Choose a reason for hiding this comment

coderabbitai bot left a comment

Choose a reason for hiding this comment

coderabbitai bot left a comment

Choose a reason for hiding this comment

coderabbitai bot Mar 6, 2025

Choose a reason for hiding this comment

coderabbitai bot Mar 6, 2025

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

uson1004 commented Mar 5, 2025 •

edited

Loading

coderabbitai bot commented Mar 5, 2025 •

edited

Loading

CodeRabbit Configuration File (`.coderabbit.yaml`)