🔀 :: (#51) packgename을 대문자에서 소문자로 변경했습니다. by kimjeakwan · Pull Request #55 · Team-MindWay/Dormitory-Manager-Android

kimjeakwan · 2025-02-05T07:07:20Z

💡 개요

packgename을 대문자에서 소문자로 변경했습니다.

📃 작업내용

packgename을 대문자에서 소문자로 변경했습니다.

🔀 변경사항

viewModel 에서 viewmodel 로 변경

🙋‍♂️ 질문사항

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

🍴 사용방법

🎸 기타

Summary by CodeRabbit

Chores
- 내부 빌드 설정의 형식 조정 및 데이터 값 업데이트를 통해 유지 관리 작업을 진행했습니다.
Refactor
- 코드 전반에 걸쳐 패키지 명칭 표준화를 적용해 일관성을 강화했습니다.
- 사용자 상태 관리 로직을 개선하여 데이터 불변성을 높였습니다.
Bug Fixes
- VERSION_CODE와 VERSION_NAME을 업데이트하여 버전 관리를 개선했습니다.
- SWIPE_REFRESH 의존성을 제거하여 관련 기능을 정리했습니다.
New Features
- 문자열 리소스의 포맷 지정자를 개선하여 인수의 순서를 명확히 했습니다.
- 로그인 화면의 상태 관리 및 제어 흐름을 향상시켜 오류 처리 및 비동기 상태 업데이트를 통합했습니다.
Documentation
- presentation/build.gradle.kts 파일의 내용이 삭제되어 빌드 구성 정보가 제거되었습니다.

coderabbitai · 2025-02-05T07:07:27Z

Walkthrough

이 풀 리퀘스트는 여러 파일에 걸쳐 형식 수정 및 패키지 네이밍 표준화를 진행합니다. 구체적으로, build.gradle.kts에 allprojects 블록을 추가하여 Kotlin 컴파일 작업을 설정하고, counters.tab에서 숫자 값을 교체하며, login 및 users 관련 파일의 패키지 이름과 import 경로를 소문자로 일관되게 수정하였습니다. 또한, UsersViewModel의 성공 상태 처리 시 데이터가 불변 리스트로 변환되도록 변경되었습니다. AuthViewModel 및 LoginScreen의 상태 관리가 개선되었으며, presentation/build.gradle.kts 파일의 내용이 삭제되었습니다.

Changes

파일(들)	변경 내용
`build.gradle.kts`	`allprojects` 블록 추가, 프로젝트 속성에 따라 컴파일러 인자 설정
`buildSrc/.../counters.tab`	값 `-6` 제거, `+5` 추가
`presentation/.../login/uiState/{LoginUiState.kt, SaveTokenUiState.kt}`	패키지 이름을 camelCase에서 소문자로 변경
`presentation/.../viewmodel/login/AuthViewModel.kt`	import 경로를 업데이트하여 최신 패키지 명 반영
`presentation/.../users/{UsersViewModel.kt, uistate/UsersUiState.kt}`	패키지 이름 수정 및 성공 상태에서 불변 리스트 사용 도입
`buildSrc/.../Dependency.kt`	상수 `SWIPE_REFRESH` 제거
`buildSrc/.../ProjectProperties.kt`	`VERSION_CODE`를 `20`으로, `VERSION_NAME`을 `"1.2.9"`로 업데이트
`presentation/build.gradle.kts`	전체 내용 삭제
`presentation/.../values/string.xml`	문자열 리소스의 포맷 지정자 업데이트
`presentation/.../login/screen/LoginScreen.kt`	`LoginRoute` 및 `LoginScreen` 함수의 시그니처 업데이트, 상태 관리 개선

Suggested labels

✨ Feature, 1️⃣ Priority: High

Suggested reviewers

xx-lvr

Poem

나는 달빛 아래를 뛰노는 토끼,
코드 숲 속에서 발자취를 남기네,
빈 줄이 춤추고 숫자가 반짝이며,
패키지 이름도 소문자로 일관되네,
불변 리스트의 빛이 별처럼 피어나,
오늘도 코드의 길을 신나게 달리리!

Tip

🌐 Web search-backed reviews and chat

We have enabled web search-based reviews and chat for all users. This feature allows CodeRabbit to access the latest documentation and information on the web.
You can disable this feature by setting web_search: false in the knowledge_base settings.
Please share any feedback in the Discord discussion.

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:

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. (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.

coderabbitai

Actionable comments posted: 1

🧹 Nitpick comments (2)

presentation/src/main/java/view/login/screen/LoginScreen.kt (2)

63-71: 함수 파라미터 네이밍을 보다 직관적으로 변경하는 것을 권장합니다.
ClickButton은 Boolean 타입임에도 불구하고 동사가 포함되지 않아 상태라는 점이 명확하지 않습니다. leIsClickLoginButton 역시 클릭 처리 함수를 의미한다는 사실을 쉽게 인지하기 어렵습니다. isButtonClicked, onButtonClick 등의 이름으로 변경하면 가독성이 한층 높아집니다.

73-94: 에러 처리 로직을 세분화하면 유지보수에 도움이 됩니다.
현재 onErrorToast에 message = 0으로 고정된 값이 전달되고 있습니다. 다양한 예외 상황을 구분하여 메시지를 알맞게 설정하거나, 별도의 에러 타입(예: enum)을 사용하는 방법을 고려해보세요.

📜 Review details

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

📥 Commits

Reviewing files that changed from the base of the PR and between 9639724 and ceb5370.

📒 Files selected for processing (1)

presentation/src/main/java/view/login/screen/LoginScreen.kt (4 hunks)

🔇 Additional comments (4)

presentation/src/main/java/view/login/screen/LoginScreen.kt (4)

12-14: 새로운 런타임 API 임포트가 적절하게 추가되었습니다.
collectAsStateWithLifecycle와 mutableStateOf 등 Jetpack Compose와 상태 관리를 위한 API 사용에 필요한 임포트가 잘 반영되었습니다.

26-33: 패키지 및 라이브러리 네이밍 구조가 일관성 있게 적용되었습니다.
DI와 상태 관찰 등에 필요한 의존성이 정상적으로 추가 및 변경되었으며, 수정된 패키지 경로(viewmodel → viewmodel.login.uistate 등)가 문제 없이 연결됩니다.

160-169: 미리보기(Preview) 구현이 적절합니다.
테스트 및 디자인 확인을 위해 필요한 파라미터들이 잘 전달되고 있어, Compose 프리뷰에서 UI를 쉽게 검증할 수 있습니다.

141-150: GAuthSigninWebView에 사용된 clientId와 redirectUri가 상수 문자열로 노출되고 있습니다.
이 값들이 실제 민감한 정보인지, 혹은 단순 텍스트 플레이스홀더인지 확인이 필요합니다. 민감한 정보라면 안전한 방식으로 관리해야 합니다.

아래 스크립트를 통해 관련 키워드가 다른 곳에서도 사용하는지 검색해 볼 수 있습니다:
✅ Verification successful

🏁 Script executed:
#!/bin/bash
rg "ghskfend|ghskfen" -A 5
Length of output: 645

플레이스홀더 문자열 사용 확인됨
스캔 결과, “ghskfend”와 “ghskfen” 문자열은 오직 presentation/src/main/java/view/login/screen/LoginScreen.kt 파일에서만 사용되고 있으며, 실제 민감한 정보라기보다는 플레이스홀더로 보입니다. 만약 이 값들이 실제 민감한 데이터라면 별도의 안전한 관리 방식(예: 환경 변수 관리, 암호화 저장 등)이 필요하겠지만, 현 시점에서는 단순 플레이스홀더로 판단됩니다.

coderabbitai · 2025-02-05T08:31:01Z

presentation/src/main/java/view/login/screen/LoginScreen.kt

+    val (ClickButton, leIsClickLoginButton) = rememberSaveable { mutableStateOf(false) }
+
+    LoginScreen(
+        modifier = modifier,
+        ClickButton = ClickButton,
+        loginUiState = loginUiState,
+        saveTokenUiState = saveTokenUiState,
+        onErrorToast = { throwable, message ->  },
+        navigateToHome = navigateToHome,
+        gAuthLogin = { code -> authViewModel.gAuthLogin(code = code)  },
+        leIsClickLoginButton = {leIsClickLoginButton(!ClickButton)}


🛠️ Refactor suggestion

rememberSaveable 상태 변수 및 함수명이 충돌되어 혼동을 유발할 수 있습니다.
동일한 이름(leIsClickLoginButton)을 상태의 Setter 함수와 파라미터로 함께 사용하고 있어, 가독성과 유지보수성에 문제가 생길 수 있습니다. Setter 함수를 명확히 구분할 수 있도록 아래처럼 변경을 제안드립니다.

- val (ClickButton, leIsClickLoginButton) = rememberSaveable { mutableStateOf(false) } + val (isLoginButtonClicked, setLoginButtonClicked) = rememberSaveable { mutableStateOf(false) } ... - leIsClickLoginButton = { leIsClickLoginButton(!ClickButton) } + leIsClickLoginButton = { setLoginButtonClicked(!isLoginButtonClicked) }

📝 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 (ClickButton, leIsClickLoginButton) = rememberSaveable { mutableStateOf(false) }

LoginScreen(

modifier = modifier,

ClickButton = ClickButton,

loginUiState = loginUiState,

saveTokenUiState = saveTokenUiState,

onErrorToast = { throwable, message -> },

navigateToHome = navigateToHome,

gAuthLogin = { code -> authViewModel.gAuthLogin(code = code) },

leIsClickLoginButton = {leIsClickLoginButton(!ClickButton)}

val (isLoginButtonClicked, setLoginButtonClicked) = rememberSaveable { mutableStateOf(false) }

LoginScreen(

modifier = modifier,

ClickButton = ClickButton,

loginUiState = loginUiState,

saveTokenUiState = saveTokenUiState,

onErrorToast = { throwable, message -> },

navigateToHome = navigateToHome,

gAuthLogin = { code -> authViewModel.gAuthLogin(code = code) },

leIsClickLoginButton = { setLoginButtonClicked(!isLoginButtonClicked) }

xx-lvr

수고하셨습니다!

🔀 ::(#55) Mypage에 통신한 것을 적용하기위한 리펙토링을 했습니다.

kimjeakwan added 6 commits February 5, 2025 16:01

:box: :: add file

2a29c55

♻️ :: packgename 이름을 대문자에서 전부 소문자로 변경

36049aa

♻️ :: packgename 이름을 대문자에서 전부 소문자로 변경

61dd8a7

♻️ :: packgename 이름을 대문자에서 전부 소문자로 변경

8d1656b

♻️ :: packgename 이름을 대문자에서 전부 소문자로 변경

e12bf36

♻️ :: packgename 이름을 대문자에서 전부 소문자로 변경

a0adfa9

kimjeakwan added 4 commits February 5, 2025 16:09

♻️ :: 공백삭제

3cf1d4c

:box: :: 빌드돌린후 파일추가

4e6a926

♻️ 의존성 변경

9639724

♻️ 쓰지않는 import문 정리

ceb5370

coderabbitai bot reviewed Feb 5, 2025

View reviewed changes

Merge branch 'develop' into feature/51-uqdate-packgename

5455009

xx-lvr approved these changes Feb 5, 2025

View reviewed changes

kimjeakwan merged commit 56857ac into develop Feb 5, 2025
2 checks passed

kimjeakwan deleted the feature/51-uqdate-packgename branch February 5, 2025 10:34

kimjeakwan added a commit that referenced this pull request Feb 19, 2025

Merge pull request #61 from Team-MindWay/feature/55-mypage-network

0a1e29a

🔀 ::(#55) Mypage에 통신한 것을 적용하기위한 리펙토링을 했습니다.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🔀 :: (#51) packgename을 대문자에서 소문자로 변경했습니다.#55

🔀 :: (#51) packgename을 대문자에서 소문자로 변경했습니다.#55
kimjeakwan merged 11 commits intodevelopfrom
feature/51-uqdate-packgename

kimjeakwan commented Feb 5, 2025 •

edited by coderabbitai bot

Loading

Uh oh!

coderabbitai bot commented Feb 5, 2025 •

edited

Loading

Chat

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (`.coderabbit.yaml`)

Documentation and Community

Uh oh!

coderabbitai bot left a comment

Uh oh!

coderabbitai bot Feb 5, 2025

Uh oh!

xx-lvr left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

kimjeakwan commented Feb 5, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

💡 개요

📃 작업내용

🔀 변경사항

🙋‍♂️ 질문사항

🍴 사용방법

🎸 기타

Summary by CodeRabbit

Uh oh!

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

Uh oh!

Walkthrough

Changes

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!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot Feb 5, 2025

Choose a reason for hiding this comment

Uh oh!

xx-lvr left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

kimjeakwan commented Feb 5, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Feb 5, 2025 •

edited

Loading

CodeRabbit Configuration File (`.coderabbit.yaml`)