Skip to content

[FEAT]: 피보호인 프로필 기능 추가 및 UI 컴포넌트 구현 - #8

Open
juhy0987 wants to merge 1 commit into
mainfrom
feature/senior-profile
Open

juhy0987 wants to merge 1 commit into
mainfrom
feature/senior-profile

Conversation

@juhy0987

Copy link
Copy Markdown
Member

No description provided.

@juhy0987
juhy0987 requested a review from Copilot August 29, 2025 11:41
@juhy0987 juhy0987 self-assigned this Aug 29, 2025
@juhy0987 juhy0987 added the enhancement New feature or request label Aug 29, 2025

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

이 PR은 피보호인(부모님) 프로필 기능을 추가하고 관련 UI 컴포넌트를 구현합니다. 기존의 단순한 홈 화면을 부모님 목록을 표시하는 기능적인 화면으로 개선하고, 새로운 프로필 화면을 추가했습니다.

  • 새로운 Person 데이터 모델과 PersonStatus enum 추가
  • 피보체인 프로필 화면 구현 (상태 변경, 연락 기능 등)
  • 홈 화면을 부모님 목록 표시 화면으로 리팩토링
  • 재사용 가능한 UI 컴포넌트들 (PersonListItem, ParentsList, HomeHeader, BottomNavigationBar) 추가

Reviewed Changes

Copilot reviewed 13 out of 16 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
gradlew.bat Gradle wrapper batch script improvements with better error handling
gradlew Updated Gradle wrapper shell script with POSIX compliance
gradle/wrapper/gradle-wrapper.properties Gradle version downgrade and configuration updates
gradle/libs.versions.toml Added Coil image loading library dependency
app/src/main/res/values/themes.xml Changed app theme to remove action bar
app/src/main/java/com/khigh/seniormap/ui/screens/PersonProfileScreen.kt New profile screen for individual persons
app/src/main/java/com/khigh/seniormap/ui/screens/HomeScreen.kt Refactored home screen to show parents list
app/src/main/java/com/khigh/seniormap/ui/model/Person.kt New data model for person and status information
app/src/main/java/com/khigh/seniormap/ui/composables/*.kt New reusable UI components
app/build.gradle.kts Added Coil dependency
Files not reviewed (3)
  • .idea/codeStyles/Project.xml: Language not supported
  • .idea/codeStyles/codeStyleConfig.xml: Language not supported
  • .idea/inspectionProfiles/Project_Default.xml: Language not supported
Comments suppressed due to low confidence (1)

app/src/main/java/com/khigh/seniormap/ui/screens/PersonProfileScreen.kt:1

  • Duplicate import statement for Icons.Default.Info. Remove one of these duplicate imports.
package com.khigh.seniormap.ui.screens

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines +8 to 10
import androidx.compose.ui.Modifier
import androidx.compose.ui.Alignment
import androidx.compose.ui.Modifier

Copilot AI Aug 29, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Duplicate import statement for Modifier. Remove the duplicate import.

Suggested change
import androidx.compose.ui.Modifier
import androidx.compose.ui.Alignment
import androidx.compose.ui.Modifier

Copilot uses AI. Check for mistakes.
Comment on lines +123 to +127
HomeScreen(
onNavigateToLogin = {},
onNavigateToProfile = {},
onNavigateToAddPerson = {},
onNavigate = {}

Copilot AI Aug 29, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Preview function is missing the required authViewModel parameter for HomeScreen. This will cause a compilation error.

Suggested change
HomeScreen(
onNavigateToLogin = {},
onNavigateToProfile = {},
onNavigateToAddPerson = {},
onNavigate = {}
// Provide a dummy AuthViewModel for preview purposes
val dummyAuthViewModel = AuthViewModel()
HomeScreen(
onNavigateToLogin = {},
onNavigateToProfile = {},
onNavigateToAddPerson = {},
onNavigate = {},
authViewModel = dummyAuthViewModel

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants