Feature/#26 navigation #27

ashwon12 · 2025-01-21T08:48:05Z

💡 Issue

Resolved: [Feature] Navigation 구조 만들기 #26

🌱 Key changes

navHost 생성
경로 지정을 관리할 수 있는 클래스 생성 ( Navigator.kt )
피처모듈에 확장함수 생성

✅ To Reviewers

route 관리를 어떻게 할까 고민하다가 .. !
확장함수를 사용하면서 app모듈을 참조하지 않기 위해 각각의 피처 안에 선언해주었습니다.
( ex @Serializable data object LoginRoute )

nia 구조가 비슷해서 참고하였습니다 ~

Summary by CodeRabbit

릴리즈 노트

새로운 기능
- 앱 내비게이션 시스템 도입
- 로그인, 회원가입, 홈, 공지사항 화면 추가
- Kotlin 직렬화 지원 추가
의존성 업데이트
- 새로운 네비게이션 컴포즈 라이브러리 통합
- Kotlin 직렬화 라이브러리 추가
개선 사항
- 앱 구조를 모듈화하여 유연성 향상
- 화면 간 전환을 위한 통합 네비게이션 메커니즘 구현

coderabbitai · 2025-01-21T08:48:11Z

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

이 풀 리퀘스트는 Android 애플리케이션의 네비게이션 구조를 개선하기 위한 포괄적인 변경 사항을 포함하고 있습니다. 주요 변경 사항은 Jetpack Compose 기반의 네비게이션 시스템 구현으로, 로그인, 회원가입, 홈, 공지사항 등 다양한 화면 간 이동을 관리하는 새로운 네비게이션 메커니즘을 도입했습니다. 빌드 구성과 의존성 관리도 함께 업데이트되었습니다.

Changes

파일	변경 요약
`app/build.gradle.kts`	디자인 시스템, 네비게이션 런타임, 컴포즈 네비게이션 의존성 추가
`app/src/main/java/com/yapp/app/official/MainActivity.kt`	네비게이터 초기화 및 `YappApp` 컴포저블로 UI 구조 변경
`app/src/main/java/com/yapp/app/official/navigation/YappNavHost.kt`	새로운 네비게이션 호스트 컴포저블 추가
`app/src/main/java/com/yapp/app/official/ui/Navigator.kt`	네비게이션 상태 관리를 위한 `NavigatorState` 클래스 도입
`app/src/main/java/com/yapp/app/official/ui/YappApp.kt`	앱 전체 UI 구조를 관리하는 `YappApp` 컴포저블 생성
기타 feature 모듈	각 기능별 네비게이션 및 화면 컴포저블 추가

Assessment against linked issues

목표	해결 여부	설명
NavHost 생성	✅
각 모듈 내 확장 함수 생성	✅

Possibly related PRs

feat: MviStore 및 샘플 코드 추가 #1: UI 관련 의존성 추가
Feature/#7 custom plugin #9: 빌드 구성 및 플러그인 구조 변경

Suggested labels

navigation, compose, feature-implementation

Suggested reviewers

jinukeu

Poem

🐰 네비게이션의 토끼 🚀
화면을 넘나드는 경로
컴포즈의 마법으로
앱의 여정을 그리다
자유롭게 흐르네! 🌈

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

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

‼️ IMPORTANT
Auto-reply has been disabled for this repository in the CodeRabbit settings. The CodeRabbit bot will not respond to your replies unless it is explicitly tagged.

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. (Beta)
@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.

jinukeu

고생하셨습니다!!!

jinukeu · 2025-01-21T09:05:09Z

feature/signup/src/main/java/com/yapp/feature/signup/navigation/SignUpNavigation.kt

+fun NavController.navigateToSignUp(navOptions: NavOptions? = null) { navigate(SignUpRoute, navOptions) }
+
+fun NavGraphBuilder.signupNavGraph(){


좋네요 👍

jinukeu · 2025-01-21T09:07:05Z

app/src/main/java/com/yapp/app/official/navigation/YappNavHost.kt

+        modifier = modifier,
+    ) {
+        loginNavGraph(
+            onSignUpClick = { navigator.navigateSignUpScreen() }


�C: NIA에서도 loginNavGraph에 onXXXClick 네이밍이 들어가나요?

nia에서는 toSignUpDestination 이런 네이밍이 들어가는데
저희는 버튼을 통해서 전환이 되어서 click 네이밍이 더 직관적일 것 같다고 생각했어요 !

아하 음 .. onSignUpClick이 SignUpButton을 클릭한 경우인가요?

넵 맞습니다~~

엇 그렇다면 Graph에서는 Screen의 동작을 모르게끔 하는게 나중에 유지보수 하기 더 좋을 것 같아요!

오호 .. 좋습니다 ! 로그인 뷰 작업하면 매개변수가 추가 될 것 같아서 그 때 같이 네이밍 반영하도록 할게요 !

jinukeu · 2025-01-21T09:09:14Z

build-logic/src/main/java/com/yapp/KotlinAndroid.kt

-    pluginManager.apply("org.jetbrains.kotlin.android")
+    with(pluginManager) {
+        apply("org.jetbrains.kotlin.android")
+        apply("org.jetbrains.kotlin.plugin.serialization")


A: 오 Nia에서도 serialization을 여기서 하나요?

앗 아녀 ㅎㅋ 컨밴션 이번에 추가하면서 느낀건데 수정이 필요로 해보입니다.. !
( register 방식 vs / .gradle.kts 플러그인 파일.)

우선 순위 높은 작업 끝나면 논의 통해서 개선하는게 좋을 것 같습니닷

좋습니다!! 일단 요렇게 고고 하시죠

ashwon12 · 2025-01-21T09:18:07Z

@coderabbitai review

coderabbitai · 2025-01-21T09:18:12Z

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

coderabbitai

Actionable comments posted: 16

♻️ Duplicate comments (1)

feature/signup/src/main/java/com/yapp/feature/signup/navigation/SignUpNavigation.kt (1)

10-18: 🧹 Nitpick (assertive)

공개 API에 대한 문서화 필요

HomeNavigation.kt와 동일한 패턴으로, 다음 항목들에 대한 KDoc 문서화가 필요합니다:

SignUpRoute 데이터 객체

navigateToSignUp 확장 함수

signupNavGraph 확장 함수

📜 Review details

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

📥 Commits

Reviewing files that changed from the base of the PR and between da98001 and 67b7dd7.

📒 Files selected for processing (17)

app/build.gradle.kts (1 hunks)
app/src/main/java/com/yapp/app/official/MainActivity.kt (2 hunks)
app/src/main/java/com/yapp/app/official/navigation/YappNavHost.kt (1 hunks)
app/src/main/java/com/yapp/app/official/ui/Navigator.kt (1 hunks)
app/src/main/java/com/yapp/app/official/ui/YappApp.kt (1 hunks)
build-logic/src/main/java/com/yapp/ComposeAndroid.kt (1 hunks)
build-logic/src/main/java/com/yapp/KotlinAndroid.kt (3 hunks)
build.gradle.kts (1 hunks)
feature/home/src/main/java/com/yapp/feature/home/HomeScreen.kt (1 hunks)
feature/home/src/main/java/com/yapp/feature/home/navigation/HomeNavigation.kt (1 hunks)
feature/login/src/main/java/com/yapp/feature/login/LoginScreen.kt (1 hunks)
feature/login/src/main/java/com/yapp/feature/login/navigation/LoginNavigation.kt (1 hunks)
feature/notice/src/main/java/com/yapp/feature/notice/NoticeScreen.kt (1 hunks)
feature/notice/src/main/java/com/yapp/feature/notice/navigation/NoticeNavigation.kt (1 hunks)
feature/signup/src/main/java/com/yapp/feature/signup/SignUpScreen.kt (1 hunks)
feature/signup/src/main/java/com/yapp/feature/signup/navigation/SignUpNavigation.kt (1 hunks)
gradle/libs.versions.toml (3 hunks)

🔇 Additional comments (12)

feature/login/src/main/java/com/yapp/feature/login/LoginScreen.kt (1)

12-19: ⚠️ Potential issue

로그인 화면 UI 요소 추가 필요

현재 구현된 로그인 화면에는 필수적인 UI 요소들이 누락되어 있습니다:

이메일/아이디 입력 필드

비밀번호 입력 필드

로그인 상태 표시

회원가입 링크

비밀번호 찾기 링크

또한 빈 onClick 핸들러는 실제 로그인 로직으로 대체되어야 합니다.

feature/home/src/main/java/com/yapp/feature/home/HomeScreen.kt (1)

15-19: onClick 핸들러 구현 및 버튼 텍스트 개선 필요

다음 사항들을 개선해주세요:

버튼의 onClick 핸들러가 비어있습니다

"Home"이라는 버튼 텍스트는 사용자에게 버튼의 목적을 명확히 전달하지 못합니다

feature/notice/src/main/java/com/yapp/feature/notice/navigation/NoticeNavigation.kt (2)

10-11: Serializable 데이터 객체의 적절한 사용을 확인했습니다.

Route 객체를 @Serializable data object로 정의한 것은 좋은 접근 방식입니다.

12-12: 네비게이션 확장 함수의 구현이 간결합니다.

NavController 확장 함수를 통해 네비게이션 로직을 캡슐화한 것이 좋습니다.

app/src/main/java/com/yapp/app/official/MainActivity.kt (1)

19-22: 네비게이션 상태 관리와 테마 적용이 잘 구현되었습니다.

rememberNavigator()를 통한 네비게이션 상태 관리가 적절합니다.

YappTheme으로 앱 전체 UI를 래핑하여 일관된 디자인 시스템 적용이 가능합니다.

app/src/main/java/com/yapp/app/official/navigation/YappNavHost.kt (1)

23-25: 이전 리뷰 의견을 반영해주세요

이전 리뷰에서 논의된 대로, onSignUpClick 대신 화면 전환 동작을 더 잘 표현하는 네이밍으로 변경하는 것이 좋겠습니다.

app/src/main/java/com/yapp/app/official/ui/Navigator.kt (1)

29-31: 사용되지 않는 프로퍼티가 있습니다

currentDestination 프로퍼티가 선언되어 있지만 사용되지 않고 있습니다. 필요하지 않다면 제거하거나, 필요하다면 활용 방안을 구현해주세요.
build-logic/src/main/java/com/yapp/KotlinAndroid.kt (2)

20-23: 🧹 Nitpick (assertive)

플러그인 구성 방식 개선 필요

이전 리뷰 코멘트에서 논의된 것처럼, 플러그인 구성 방식에 대한 개선이 필요해 보입니다. 현재는 직접 apply를 사용하고 있지만, 빌드 로직의 일관성을 위해 .gradle.kts 플러그인 파일 방식으로 전환을 고려해볼 수 있습니다.

51-54: 직렬화 라이브러리 버전 검증 필요

직렬화 라이브러리 의존성이 추가되었습니다. 프로젝트의 안정성을 위해 버전 호환성을 확인해주세요.

다음 스크립트로 kotlinx.serialization 라이브러리의 최신 버전과 호환성을 확인할 수 있습니다:

✅ Verification successful

kotlinx.serialization 라이브러리 버전 검증 완료

프로젝트에서 사용 중인 kotlinx.serialization 1.8.0 버전은 최신 안정 버전이며, Kotlin 2.0.0과 완벽하게 호환됩니다. 버전 호환성에 문제가 없습니다.
🏁 Scripts executed

The following scripts were executed for the analysis:

Script:
#!/bin/bash
# kotlinx.serialization 라이브러리의 최신 버전 확인
curl -s "https://api.github.com/repos/Kotlin/kotlinx.serialization/releases/latest" | jq -r '.tag_name'

# Kotlin 버전과의 호환성 매트릭스 확인
rg "kotlin.*version" -g "*.{gradle,kts,properties}" 
Length of output: 162

Script:
#!/bin/bash
# Find and check version catalog
fd "libs.versions.toml" --type f --exec cat {}

# Search for version declarations in gradle files with different patterns
rg "kotlin|serialization" -g "*.{gradle,gradle.kts,toml,properties}" -A 2 -B 2

# Check buildSrc directory for version declarations
fd . "buildSrc" --type f --exec cat {}
Length of output: 7918
build.gradle.kts (1)

8-8: 변경사항 승인

직렬화 플러그인이 적절하게 추가되었습니다.
gradle/libs.versions.toml (2)

5-5: Kotlin 직렬화 라이브러리 버전 업데이트 권장

현재 사용 중인 kotlinx.serialization.json 버전(1.8.0)이 최신 버전이 아닐 수 있습니다. 최신 안정 버전 사용을 권장드립니다.

다음 스크립트로 최신 버전을 확인해보세요:

21-23: ⚠️ Potential issue

네비게이션 버전 정의 개선 필요

중복된 버전 정의가 있습니다:

navigationCompose와 navigationComposeVersion이 유사한 목적으로 사용됨

버전 번호가 현재 안정 버전보다 높아 보입니다:

2.8.4, 2.8.5는 현재 존재하지 않는 버전일 수 있음

다음과 같이 개선하는 것을 추천드립니다:

네비게이션 관련 버전을 하나로 통일

최신 안정 버전 사용

버전 검증을 위해 다음 스크립트를 실행해보세요:

✅ Verification successful

네비게이션 라이브러리 버전 통일 필요

버전 번호는 모두 유효한 안정 버전이나, 다음 개선사항을 제안드립니다:

중복 버전 정의 제거:

navigationCompose와 navigationComposeVersion 중 하나만 사용

모든 네비게이션 컴포넌트를 동일 버전으로 통일:

현재 2.8.4와 2.8.5가 혼용되어 있음

모두 2.8.5로 통일 권장
🏁 Scripts executed

The following scripts were executed for the analysis:

Script:
#!/bin/bash
# 최신 안정 버전 확인
curl -s "https://dl.google.com/dl/android/maven2/androidx/navigation/navigation-compose/maven-metadata.xml" | grep -oP "(?<=<version>)[^<]+(?=</version>)" | tail -n 1
Length of output: 180

Script:
#!/bin/bash
# Get all available versions
curl -s "https://dl.google.com/dl/android/maven2/androidx/navigation/navigation-compose/maven-metadata.xml" | grep -oP "(?<=<version>)[^<]+(?=</version>)" | sort -V
Length of output: 1186

feature/login/src/main/java/com/yapp/feature/login/LoginScreen.kt

app/src/main/java/com/yapp/app/official/ui/YappApp.kt

feature/home/src/main/java/com/yapp/feature/home/navigation/HomeNavigation.kt

feature/notice/src/main/java/com/yapp/feature/notice/navigation/NoticeNavigation.kt

coderabbitai · 2025-01-21T09:57:18Z

build-logic/src/main/java/com/yapp/ComposeAndroid.kt

+            add("implementation", libs.findLibrary("androidx.material3").get())
+            add("implementation", libs.findLibrary("androidx.navigation.compose").get())


⚠️ Potential issue

중복된 의존성을 제거해주세요

androidx.material3 의존성이 중복 선언되어 있습니다. 23번 라인을 제거해주세요.

app/src/main/java/com/yapp/app/official/ui/Navigator.kt

coderabbitai · 2025-01-21T09:57:18Z

app/build.gradle.kts

+    implementation(libs.androidx.navigation.runtime.ktx)
+    implementation(libs.navigation.compose)


🛠️ Refactor suggestion

네비게이션 라이브러리 버전 불일치 확인 필요

libs.versions.toml에 정의된 네비게이션 라이브러리 버전들 사이에 불일치가 있습니다:

navigationCompose: 2.8.4

navigationRuntimeKtx: 2.8.5

navigationComposeVersion: 2.8.5

버전을 통일하여 잠재적인 호환성 문제를 방지하는 것이 좋습니다.

coderabbitai · 2025-01-21T09:57:18Z

gradle/libs.versions.toml


 androidx-core-ktx = { group = "androidx.core", name = "core-ktx", version.ref = "coreKtx" }
 androidx-appcompat = { group = "androidx.appcompat", name = "appcompat", version.ref = "appcompat" }
 material = { group = "com.google.android.material", name = "material", version.ref = "material" }
 androidx-lifecycle-runtime-ktx = { group = "androidx.lifecycle", name = "lifecycle-runtime-ktx", version.ref = "lifecycleRuntimeKtx" }
 androidx-activity-compose = { group = "androidx.activity", name = "activity-compose", version.ref = "activityCompose" }
+androidx-navigation-compose = { module = "androidx.navigation:navigation-compose", version.ref = "navigationCompose" }


🛠️ Refactor suggestion

중복 네비게이션 의존성 제거 필요

동일한 기능의 네비게이션 의존성이 여러 번 정의되어 있습니다:

androidx-navigation-compose

navigation-compose

하나의 의존성만 사용하도록 정리가 필요합니다.

Also applies to: 55-56

ashwon12 added 4 commits January 21, 2025 17:36

gradle/#26 : navigation 의존성 추가

0e2ae8a

feat/#26 : navigator 상태 추가

e013975

feat/#26 : feature 모듈에 navigation 함수 생성

fc3f01c

feat/#26 : navHost 생성

67b7dd7

ashwon12 added the Feature 기능 추가, 개발 label Jan 21, 2025

ashwon12 requested a review from jinukeu January 21, 2025 08:48

ashwon12 self-assigned this Jan 21, 2025

jinukeu reviewed Jan 21, 2025

View reviewed changes

jinukeu approved these changes Jan 21, 2025

View reviewed changes

ashwon12 added the CodeRabbit CodeRabbit 코드 리뷰 요청 label Jan 21, 2025

coderabbitai bot reviewed Jan 21, 2025

View reviewed changes

gradle/#26 : 중복 코드 제거

50a7980

ashwon12 merged commit 81ab12d into develop Jan 21, 2025
1 check passed

ashwon12 deleted the feature/#26-navigation branch January 21, 2025 14:59

coderabbitai bot mentioned this pull request Feb 10, 2025

Feature/#63 공지사항 View #80

Merged

This was referenced Mar 25, 2025

Feature/#94 notice detail #95

Merged

Feature/#98 강제 업데이트, 앱 로고, 스플래시 스크린 #108

Merged

This was referenced May 5, 2025

Feature/#156 홈 UI 수정 #157

Merged

Feature/#170 notice schedule minor qa #171

Merged

coderabbitai bot mentioned this pull request May 24, 2025

[Feature] 홈 화면 최근 출석내역 #201

Merged

		fun NavController.navigateToSignUp(navOptions: NavOptions? = null) { navigate(SignUpRoute, navOptions) }

		fun NavGraphBuilder.signupNavGraph(){

		add("implementation", libs.findLibrary("androidx.material3").get())
		add("implementation", libs.findLibrary("androidx.navigation.compose").get())

		implementation(libs.androidx.navigation.runtime.ktx)
		implementation(libs.navigation.compose)

Feature/#26 navigation #27

Feature/#26 navigation #27

Uh oh!

Conversation

ashwon12 commented Jan 21, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

💡 Issue

🌱 Key changes

✅ To Reviewers

Summary by CodeRabbit

릴리즈 노트

Uh oh!

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

Uh oh!

Review skipped

Walkthrough

Changes

Assessment against linked issues

Possibly related PRs

Suggested labels

Suggested reviewers

Poem

Chat

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (.coderabbit.yaml)

Documentation and Community

Uh oh!

jinukeu left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jinukeu Jan 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ashwon12 commented Jan 21, 2025

Uh oh!

coderabbitai bot commented Jan 21, 2025

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

coderabbitai bot Jan 21, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

coderabbitai bot Jan 21, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot Jan 21, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

ashwon12 commented Jan 21, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Jan 21, 2025 •

edited

Loading

CodeRabbit Configuration File (`.coderabbit.yaml`)

jinukeu Jan 21, 2025 •

edited

Loading