🔀 :: (#202) - 박람회 생성 화면를 퍼블리싱 했습니다.

[audgns10]

💡 개요

• 박람회 생성 화면 퍼블리싱이 필요해보였습니다.

📃 작업내용

• 박람회 생성 화면 퍼블리싱을 하였습니다.

  • 여러번 사용되는 컴포넌트가 존재하여 Feature 모듈 안의 컴포넌트 페키지가 아닌 Design-System의 TextField 컴포넌트로 옮겼습니다.

  • 네비게이션바가 아이템과 곂치는 문제가 존재하여 paddingValues를 적용하여 해결하였습니다.

  • 해당 화면의 네비게이션 세팅을 하였습니다.

  • 2024-11-14.11.10.56.mov

🔀 변경사항

• add ExpoCreateScreen
• chore Move The Feature Component To The Design System Component

🙋‍♂️ 질문사항

• 개선할 점, 오타, 코드에 이상한 부분이 있다면 Comment 달아주세요.

🍴 사용방법

• x

🎸 기타

-x

Summary by CodeRabbit

• 새로운 기능
  • Expo 생성 화면을 위한 새로운 내비게이션 경로 및 기능 추가.
  • Expo 생성 이벤트를 위한 사용자 인터페이스 및 이미지 선택 기능 추가.
  • 텍스트 필드에 위치 아이콘 버튼 통합.
  • 내비게이션 그래프에 새로운 화면 추가.
• 버그 수정
  • 기존 UI 컴포넌트를 새로운 컴포넌트로 교체하여 입력 처리 개선.
• 문서화
  • 코드 내 주석을 통해 변경 사항에 대한 설명 유지.

[coderabbitai]

Walkthrough

이 변경 사항은 ExpoNavHost.kt, ExpoApp.kt, ExpoAppState.kt, HomeNavigation.kt, ExpoCreateScreen.kt 및 HomeDetailModifyScreen.kt 파일에서 내비게이션 기능을 향상시키고 새로운 UI 구성 요소를 추가하는 데 중점을 둡니다. 새로운 내비게이션 경로와 화면이 추가되었으며, 기존 구성 요소가 새로운 텍스트 필드 구성 요소로 대체되었습니다. 전체적으로 사용자 경험을 개선하고 내비게이션 흐름을 확장하는 방향으로 진행되었습니다.

Changes

파일 경로	변경 요약
app/src/main/java/com/school_of_company/expo_android/navigation/ExpoNavHost.kt	expoCreateScreen, homeRoute 추가 및 내비게이션 로직 수정.
app/src/main/java/com/school_of_company/expo_android/ui/ExpoApp.kt	expoCreateRoute 추가 및 topLevelDestinationRoute 배열 업데이트. Scaffold의 패딩 적용.
app/src/main/java/com/school_of_company/expo_android/ui/ExpoAppState.kt	navigateToExpoCreate 추가 및 내비게이션 기능 향상.
core/design-system/src/main/java/com/school_of_company/design_system/component/textfield/ExpoAddTextField.kt	패키지 변경 및 함수 이름 변경.
core/design-system/src/main/java/com/school_of_company/design_system/component/textfield/ExpoTextField.kt	ExpoLocationIconTextField 추가.
feature/home/src/main/java/com/school_of_company/home/navigation/HomeNavigation.kt	expoCreateRoute 및 navigateToExpoCreate 추가.
feature/home/src/main/java/com/school_of_company/home/view/ExpoCreateScreen.kt	ExpoCreateRoute 추가 및 이미지 선택 기능 구현.
feature/home/src/main/java/com/school_of_company/home/view/HomeDetailModifyScreen.kt	ExpoAddTextField, ExpoLocationIconTextField로 구성 요소 변경.
feature/home/src/main/java/com/school_of_company/home/view/component/HomeDetailModifyIconTextField.kt	HomeDetailModifyIconTextField 삭제.
settings.gradle.kts	excludedTaskNames의 순서 변경.

Possibly related PRs

• 🔀 :: (#9) - 많은 모듈이 존재할것을 생각해 버전 카탈로그를 사용한 Convention Plugin으로 의존성을 관리하는 것이 효율적일 것 같아 반영했습니다. #14: ExpoNavHost.kt의 내비게이션 로직 수정과 관련된 PR.
• 🔀 :: (#130) - 스플래시 화면을 구현하였습니다. #131: homeRoute 업데이트와 관련된 PR.
• 🔀 :: (#138) - NavHost에서 화면 전환시 애니메이션을 추가하여 화면 전환을 매끄럽게 하였습니다. #139: ExpoNavHost의 애니메이션 전환 개선과 관련된 PR.
• 🔀 :: (#164) - 회원가입을 할 때 사용하는 문자인증을 구현하고, 몇몇 세부적인 사항을 수정 및 반영하였습니다. #165: 사용자 등록 프로세스 향상과 관련된 PR.
• 🔀 :: (#193) - 박람회 -> 프로그램 버튼 클릭시 나와야하는 화면을 퍼블리싱하였습니다. #196: 내비게이션 기능 향상과 관련된 PR.
• 🔀 :: (#197) - 박람회 -> 상세보기 -> 조회하기 버튼을 눌렀을때 나오는 참가자 명단 화면을 퍼블리싱 하였습니다. #198: 참가자 관리에 대한 내비게이션 업데이트와 관련된 PR.
• 🔀 :: (#199) - 박람회 -> 박람회 상세 -> 수정하기 화면을 퍼블리싱하였습니다. #201: 수정 화면 게시와 관련된 PR.

Suggested labels

1️⃣ Priority: High, ✨ Feature

Suggested reviewers

• Chaejongin12
• diejdkll
• kimjeakwan

🐇 새로운 경로가 열리고,
내비게이션이 더 밝아져요.
텍스트 필드가 새로워,
사용자 경험을 더해줘요.
모두 함께 뛰어놀아요,
변화의 순간을 즐겨요! 🌟

---

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

🪧 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 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

🧹 Outside diff range and nitpick comments (13)

feature/home/src/main/java/com/school_of_company/home/navigation/HomeNavigation.kt (1)

143-150: 함수 문서화 추가를 제안드립니다.

함수 구현은 적절하나, 다음 사항들의 추가를 고려해주세요:

• 함수의 목적과 역할에 대한 KDoc 문서화
• onExpoCreateClick 파라미터의 용도 설명

다음과 같은 문서화를 추가하는 것을 제안드립니다:

+/**
+ * Expo 생성 화면을 네비게이션 그래프에 추가합니다.
+ *
+ * @param onExpoCreateClick Expo 생성 버튼 클릭 시 실행될 콜백
+ */
fun NavGraphBuilder.expoCreateScreen(
    onExpoCreateClick: () -> Unit
) {

core/design-system/src/main/java/com/school_of_company/design_system/component/textfield/ExpoAddTextField.kt (2)

29-36: KDoc 문서화 추가 필요

컴포넌트의 재사용성을 높이기 위해 각 매개변수의 용도와 예상되는 동작을 설명하는 KDoc 문서를 추가하는 것이 좋습니다.

다음과 같은 문서화를 제안합니다:

+/**
+ * 텍스트 필드를 동적으로 추가/제거할 수 있는 입력 컴포넌트
+ *
+ * @param modifier 컴포넌트의 수정자
+ * @param placeHolder 빈 텍스트 필드에 표시될 안내 텍스트
+ * @param trainingTextFieldList 현재 표시된 텍스트 필드들의 값 목록
+ * @param onValueChange 텍스트 필드 값이 변경될 때 호출되는 콜백 (인덱스, 새로운 값)
+ * @param onAddTextField 새로운 텍스트 필드 추가 버튼 클릭시 호출되는 콜백
+ * @param onRemoveTextField 텍스트 필드 제거 버튼 클릭시 호출되는 콜백 (제거할 인덱스)
+ */

---

Line range hint 37-119: 접근성 및 입력 유효성 검사 개선 필요

현재 구현에 다음 사항들을 추가하면 좋을 것 같습니다:

1. 텍스트 입력 제한 (최대 길이)
2. 오류 상태 표시
3. 접근성 레이블

다음과 같은 개선사항을 제안합니다:

 BasicTextField(
     value = text,
     onValueChange = { newState ->
-        onValueChange(index, newState)
+        if (newState.length <= MAX_LENGTH) {
+            onValueChange(index, newState)
+        }
     },
     textStyle = typography.bodyRegular2,
     cursorBrush = SolidColor(colors.main),
+    modifier = Modifier
+        .weight(1f)
+        .padding(end = 8.dp)
+        .semantics {
+            contentDescription = "$placeHolder ${index + 1}번째 입력창"
+        },
-    modifier = Modifier
-        .weight(1f)
-        .padding(end = 8.dp)
 )

상수 추가:

private const val MAX_LENGTH = 100 // 적절한 최대 길이로 조정

app/src/main/java/com/school_of_company/expo_android/ui/ExpoApp.kt (1)

70-75: Scaffold의 padding 처리가 개선되었습니다.

Box와 padding을 추가하여 네비게이션 바와 콘텐츠 간의 겹침 문제를 해결한 것이 좋습니다. 다만, 가독성을 위해 다음과 같이 개선하면 좋을 것 같습니다:

-            Box(modifier = Modifier.padding(paddingValues =
-            paddingValues)) {
+            Box(
+                modifier = Modifier.padding(paddingValues = paddingValues)
+            ) {
                 ExpoNavHost(appState = appState)
             }

app/src/main/java/com/school_of_company/expo_android/navigation/ExpoNavHost.kt (1)

Line range hint 68-80: 애니메이션 로직 개선이 필요합니다.

다음과 같은 개선사항을 제안드립니다:

1. 애니메이션 지속 시간을 상수로 분리하여 재사용성과 유지보수성을 높이는 것이 좋습니다.
2. homeRoute 체크 로직을 더 명확하게 표현할 수 있습니다.

다음과 같이 수정하는 것을 추천드립니다:

+ private const val FORWARD_ANIMATION_DURATION = 500
+ private const val BACK_ANIMATION_DURATION = 350

  enterTransition = {
-     if (targetState.destination.route != homeRoute) {
+     val isHomeRoute = targetState.destination.route == homeRoute
+     if (!isHomeRoute) {
          slideIntoContainer(
              AnimatedContentTransitionScope.SlideDirection.Left,
-             animationSpec = tween(durationMillis = 500)
+             animationSpec = tween(durationMillis = FORWARD_ANIMATION_DURATION)
          )
      } else {
          EnterTransition.None
      }
  }

feature/home/src/main/java/com/school_of_company/home/view/HomeDetailModifyScreen.kt (4)

Line range hint 342-359: 연수 종류 입력 필드의 상태 관리 개선이 필요합니다.

trainingTextState가 Screen 레벨에서 관리되고 있습니다. ViewModel로 상태 관리를 이동하는 것이 좋습니다.

다음과 같이 수정하는 것을 제안드립니다:

- var trainingTextState by rememberSaveable { mutableStateOf(listOf("")) }
+ val trainingTextState by viewModel.training_types.collectAsStateWithLifecycle()

ViewModel에 다음 코드를 추가해주세요:

private val _training_types = MutableStateFlow(listOf(""))
val training_types = _training_types.asStateFlow()

fun updateTrainingType(index: Int, value: String) {
    _training_types.update { current ->
        current.toMutableList().apply {
            set(index, value)
        }
    }
}

fun addTrainingType() {
    _training_types.update { it + "" }
}

fun removeTrainingType(index: Int) {
    _training_types.update { current ->
        current.toMutableList().apply {
            removeAt(index)
        }
    }
}

---

Line range hint 366-372: 위치 웹훅 구현이 필요합니다.

Location Web Hook이 TODO 주석으로 남아있습니다.

위치 검색 기능 구현에 도움이 필요하시다면 말씀해 주세요. GitHub 이슈를 생성하여 구현을 도와드리겠습니다.

---

Line range hint 385-394: 버튼 활성화 조건 로직 개선이 필요합니다.

현재 버튼 상태 로직이 너무 복잡하고 가독성이 떨어집니다.

다음과 같이 개선하는 것을 제안드립니다:

- state = if (imageUri.isNullOrEmpty() && modifyTitleState.isEmpty() && startedDateState.isEmpty() && endedDateState.isEmpty() && introduceTitleState.isEmpty() && addressState.isEmpty() && locationState.isEmpty() && trainingTextState.isEmpty()) ButtonState.Enable else ButtonState.Disable,
+ state = determineButtonState(
+     imageUri = imageUri,
+     modifyTitleState = modifyTitleState,
+     startedDateState = startedDateState,
+     endedDateState = endedDateState,
+     introduceTitleState = introduceTitleState,
+     addressState = addressState,
+     locationState = locationState,
+     trainingTextState = trainingTextState
+ ),

private fun determineButtonState(
    imageUri: String?,
    modifyTitleState: String,
    startedDateState: String,
    endedDateState: String,
    introduceTitleState: String,
    addressState: String,
    locationState: String,
    trainingTextState: List<String>
): ButtonState {
    val isAnyFieldEmpty = listOf(
        imageUri.isNullOrEmpty(),
        modifyTitleState.isEmpty(),
        startedDateState.isEmpty(),
        endedDateState.isEmpty(),
        introduceTitleState.isEmpty(),
        addressState.isEmpty(),
        locationState.isEmpty(),
        trainingTextState.isEmpty()
    ).any { it }
    
    return if (isAnyFieldEmpty) ButtonState.Disable else ButtonState.Enable
}

---

Line range hint 390-392: Expo 수정 콜백 구현이 필요합니다.

Expo Modify Callback이 TODO 주석으로 남아있습니다.

수정 기능 구현에 도움이 필요하시다면 말씀해 주세요. GitHub 이슈를 생성하여 구현을 도와드리겠습니다.

core/design-system/src/main/java/com/school_of_company/design_system/component/textfield/ExpoTextField.kt (1)

382-436: Preview 함수 추가를 제안드립니다.

새로 추가된 ExpoLocationIconTextField 컴포넌트의 Preview 함수가 누락되었습니다. 디자인 시스템 컴포넌트의 일관성을 위해 Preview 함수 추가를 권장드립니다.

@Preview
@Composable
fun ExpoLocationIconTextFieldPreview() {
    Box(
        modifier = Modifier
            .fillMaxSize()
            .background(Color.White)
            .padding(16.dp)
    ) {
        ExpoLocationIconTextField(
            placeholder = "위치를 입력하세요",
            isDisabled = false,
            onValueChange = {},
            onButtonClicked = {},
            value = ""
        )
    }
}

feature/home/src/main/java/com/school_of_company/home/view/ExpoCreateScreen.kt (3)

255-263: 변수 이름의 일관성 유지 필요

modifyTitleState와 같은 변수명은 수정 화면에 적합하며, 생성 화면에서는 titleState로 사용하는 것이 보다 명확합니다. 이는 코드의 가독성을 높이고 유지 보수를 용이하게 합니다.

관련 변수와 함수명을 다음과 같이 변경하는 것을 제안합니다:

- val modifyTitleState by viewModel.modify_title.collectAsStateWithLifecycle()
+ val titleState by viewModel.title.collectAsStateWithLifecycle()

- onModifyTitleChange: (String) -> Unit,
+ onTitleChange: (String) -> Unit,

코드 전반에 걸쳐 해당 변수명과 함수명을 일관되게 변경해주세요.

---

367-367: 버튼 텍스트 수정 필요

현재 버튼의 텍스트가 "수정완료"로 설정되어 있습니다. 생성 화면에서는 "등록하기"와 같은 텍스트가 더 적절합니다. 사용자에게 정확한 액션을 전달하기 위해 텍스트를 변경하는 것이 좋습니다.

다음과 같이 수정하는 것을 제안합니다:

- text = "수정완료",
+ text = "등록하기",

---

371-371: TODO 주석 내용 수정 필요

주석에 "Expo Modify CallBack"이라고 되어 있는데, 생성 화면이므로 "Expo Create CallBack"으로 수정하는 것이 맞습니다. 이는 코드의 명확성을 높입니다.

다음과 같이 수정하는 것을 제안합니다:

- /* todo : Expo Modify CallBack */
+ /* todo : Expo Create CallBack */

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Reviewing files that changed from the base of the PR and between 1824b06 and 6b21b84.

📒 Files selected for processing (10)

• app/src/main/java/com/school_of_company/expo_android/navigation/ExpoNavHost.kt (4 hunks)
• app/src/main/java/com/school_of_company/expo_android/ui/ExpoApp.kt (5 hunks)
• app/src/main/java/com/school_of_company/expo_android/ui/ExpoAppState.kt (2 hunks)
• core/design-system/src/main/java/com/school_of_company/design_system/component/textfield/ExpoAddTextField.kt (2 hunks)
• core/design-system/src/main/java/com/school_of_company/design_system/component/textfield/ExpoTextField.kt (3 hunks)
• feature/home/src/main/java/com/school_of_company/home/navigation/HomeNavigation.kt (4 hunks)
• feature/home/src/main/java/com/school_of_company/home/view/ExpoCreateScreen.kt (1 hunks)
• feature/home/src/main/java/com/school_of_company/home/view/HomeDetailModifyScreen.kt (3 hunks)
• feature/home/src/main/java/com/school_of_company/home/view/component/HomeDetailModifyIconTextField.kt (0 hunks)
• settings.gradle.kts (1 hunks)

💤 Files with no reviewable changes (1)

• feature/home/src/main/java/com/school_of_company/home/view/component/HomeDetailModifyIconTextField.kt

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

• settings.gradle.kts

🔇 Additional comments (10)

app/src/main/java/com/school_of_company/expo_android/ui/ExpoAppState.kt (1)

18-18: LGTM! 새로운 네비게이션 기능이 적절히 추가되었습니다.

네비게이션 관련 import가 기존 패턴과 일관성 있게 추가되었습니다.

feature/home/src/main/java/com/school_of_company/home/navigation/HomeNavigation.kt (3)

7-7: 새로운 ExpoCreateRoute 임포트가 적절히 추가되었습니다.

기존 임포트 패턴을 따르고 있으며, 새로운 화면 구현에 필요한 컴포넌트입니다.

---

23-23: 라우트 상수가 일관된 패턴으로 정의되었습니다.

기존 라우트 상수들과 동일한 네이밍 컨벤션을 따르고 있습니다.

---

53-55: 네비게이션 확장 함수가 적절히 구현되었습니다.

기존 네비게이션 함수들과 동일한 패턴으로 구현되어 있으며, 옵셔널 네비게이션 옵션을 올바르게 처리하고 있습니다.

core/design-system/src/main/java/com/school_of_company/design_system/component/textfield/ExpoAddTextField.kt (1)

1-1: 컴포넌트 사용처 확인 필요

feature 모듈에서 design-system 모듈로 이동된 컴포넌트이므로, 모든 사용처에서 올바르게 임포트되고 있는지 확인이 필요합니다.

✅ Verification successful

Let me verify the actual usage of this component in the files where it's imported.

---

컴포넌트 이동이 정상적으로 완료되었습니다

모든 사용처에서 새로운 패키지 경로(com.school_of_company.design_system.component.textfield.ExpoAddTextField)를 올바르게 참조하고 있으며, 컴포넌트가 정상적으로 사용되고 있습니다.

• HomeDetailModifyScreen.kt와 ExpoCreateScreen.kt 두 파일 모두 design-system 모듈의 새로운 경로에서 컴포넌트를 올바르게 임포트하고 있습니다.
• 각 파일에서 ExpoAddTextField 컴포넌트가 동일한 방식으로 사용되고 있습니다.

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Description: Verify the component usage across the codebase

# Check old import usage
rg "import.*HomeDetailModifyAddTextField"

# Check new import usage
rg "import.*ExpoAddTextField"

Length of output: 399

---

Script:

#!/bin/bash
# Check actual usage of ExpoAddTextField in the files
rg "ExpoAddTextField" feature/home/src/main/java/com/school_of_company/home/view/HomeDetailModifyScreen.kt -A 2
rg "ExpoAddTextField" feature/home/src/main/java/com/school_of_company/home/view/ExpoCreateScreen.kt -A 2

Length of output: 1065

app/src/main/java/com/school_of_company/expo_android/ui/ExpoApp.kt (1)

4-4: 새로운 import 구문이 적절히 추가되었습니다.

Box와 padding 관련 import가 UI 구조 변경에 맞게 잘 추가되었으며, expoCreateRoute의 추가도 적절합니다.

Also applies to: 6-6, 15-15, 26-26

app/src/main/java/com/school_of_company/expo_android/navigation/ExpoNavHost.kt (2)

14-20: LGTM! 새로운 화면에 필요한 import문이 잘 추가되었습니다.

새로운 Expo 생성 화면과 홈 라우트에 필요한 import문이 적절하게 추가되었습니다.

---

153-156: Expo 생성 후 네비게이션 흐름을 검증해주세요.

Expo 생성 화면에서 홈으로 돌아가는 네비게이션 흐름이 적절한지 확인이 필요합니다. 생성 완료/취소 시나리오에 따라 다른 네비게이션이 필요할 수 있습니다.

✅ Verification successful

Expo 생성 화면의 네비게이션 흐름이 적절하게 구현되어 있습니다.

Expo 생성 화면의 네비게이션 구현을 검토한 결과:

• ExpoCreateRoute 컴포넌트는 생성 완료 시 onExpoCreateClick 콜백을 통해 홈 화면으로 이동
• ExpoNavHost에서 navigateToHome을 통해 홈 화면으로의 네비게이션이 올바르게 연결
• 취소 시나리오는 Android의 기본 뒤로가기를 사용하여 처리

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Description: Verify navigation flow implementation in ExpoCreateScreen

# Check ExpoCreateScreen implementation for navigation handling
ast-grep --pattern 'fun ExpoCreateScreen($_, onExpoCreateClick: () -> Unit) {
  $$$
}'

# Check if there are any other navigation-related functions for ExpoCreateScreen
rg -A 5 "ExpoCreateScreen.*navigation"

Length of output: 129

---

Script:

#!/bin/bash
# Search for ExpoCreateScreen composable function
ast-grep --pattern 'composable fun ExpoCreateScreen($$$)'

# Find ExpoCreateScreen usage and navigation patterns
rg "ExpoCreateScreen" -A 5

# Look for navigation related to expo creation
rg "navigateToHome|ExpoCreate" -A 5

Length of output: 23660

feature/home/src/main/java/com/school_of_company/home/view/HomeDetailModifyScreen.kt (1)

56-57: Design System 컴포넌트로의 마이그레이션이 적절히 이루어졌습니다.

Feature 모듈의 컴포넌트들이 Design System의 재사용 가능한 컴포넌트로 잘 이동되었습니다.

core/design-system/src/main/java/com/school_of_company/design_system/component/textfield/ExpoTextField.kt (1)

14-14: 새로운 import 구문이 적절하게 추가되었습니다.

필요한 컴포넌트들이 올바르게 import 되었습니다.

Also applies to: 19-19, 42-42

[coderabbitai]

Actionable comments posted: 2

🧹 Outside diff range and nitpick comments (2)

feature/home/src/main/java/com/school_of_company/home/view/ExpoCreateScreen.kt (2)

351-351: TODO 항목 구현 지원

다음 TODO 항목들의 구현이 필요합니다:

1. Location Web Hook
2. Expo Modify CallBack

이러한 기능의 구현을 도와드릴까요? GitHub 이슈를 생성하여 추적할 수 있습니다.

Also applies to: 371-371

---

347-353: 위치 검색 기능 개선 필요

현재 위치 입력 필드는 수동 입력만 가능합니다. 사용자 경험 향상을 위해 위치 검색 API 연동이 필요합니다.

1. Kakao 로컬 API나 Naver Maps API를 활용하여 주소 검색 기능 추가
2. 검색 결과를 드롭다운 목록으로 표시
3. 선택된 주소의 위도/경도 정보도 함께 저장

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Reviewing files that changed from the base of the PR and between 6b21b84 and 21babd5.

📒 Files selected for processing (1)

• feature/home/src/main/java/com/school_of_company/home/view/ExpoCreateScreen.kt (1 hunks)

🔇 Additional comments (1)

feature/home/src/main/java/com/school_of_company/home/view/ExpoCreateScreen.kt (1)

135-135: 이전 리뷰 코멘트가 여전히 유효합니다

빈 리스트로 초기화하는 것이 더 적절합니다.

[coderabbitai]

⚠️ Potential issue

날짜 형식 검증 로직 추가 필요

현재 날짜 형식에 대한 안내만 있고 실제 검증이 없습니다. 잘못된 형식 입력 시 사용자가 즉각적인 피드백을 받을 수 없습니다.

다음과 같은 날짜 검증 함수 추가를 제안합니다:

private fun isValidDateFormat(date: String): Boolean {
    return try {
        val formatter = SimpleDateFormat("yyyy.MM.dd", Locale.getDefault())
        formatter.isLenient = false
        formatter.parse(date)
        true
    } catch (e: ParseException) {
        false
    }
}

그리고 TextField에서 다음과 같이 사용:

 LimitedLengthTextField(
     label = "모집기간",
     textState = startedDateState,
     placeholder = "시작일",
-    isError = false,
+    isError = startedDateState.isNotEmpty() && !isValidDateFormat(startedDateState),
     updateTextValue = onStartedDateChange,
     modifier = Modifier.weight(1f)
 )