Skip to content

Feature/policy api#64

Merged
haeni82 merged 2 commits into
developfrom
feature/policy-api
Mar 23, 2026
Merged

Feature/policy api#64
haeni82 merged 2 commits into
developfrom
feature/policy-api

Conversation

@haeni82
Copy link
Copy Markdown
Contributor

@haeni82 haeni82 commented Mar 23, 2026

이슈

  • closed #이슈번호

✔️ 체크리스트

  • : Merge할 브랜치를 확인해 주세요.

🔍 작업 내용

⚠️ 주의 사항 / 기타

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능

    • 권한 관리 화면에서 동일한 이름의 사용자를 구분하기 위해 휴대폰 번호 뒷자리 표시 추가
  • 버그 수정

    • 개인정보 보호 권한 판정 로직 개선
    • 자신의 데이터 표시 조건 정정
  • 스타일

    • 헤더 제목 수직 정렬 개선
    • 아이콘 버튼 크기 조정
  • 성능 개선

    • 데이터 동기화 주기 최적화로 성능 개선

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 23, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: f6992db7-cbe3-431b-b31e-a8fb18d9e88e

📥 Commits

Reviewing files that changed from the base of the PR and between 8e0f1c6 and 51ad9b0.

📒 Files selected for processing (6)
  • src/api/services/familyService.ts
  • src/components/Header.tsx
  • src/page/Detail/DetailPage.tsx
  • src/page/Detail/components/AppUsageChart.tsx
  • src/page/Main/MainPage.tsx
  • src/page/Policy/components/Permisssion.tsx

📝 Walkthrough

Walkthrough

이 PR은 Permission 인터페이스에 is_enable 필드를 추가하고, 프라이버시 권한 감지 로직을 업데이트하며, 여러 페이지에서 폴링 간격을 1000ms에서 10000ms로 증가시킵니다. 또한 AppUsageChart의 자신 데이터 표시 동작을 개선하고, 권한 UI에 전화번호 구분 표시를 추가합니다.

Changes

Cohort / File(s) Summary
권한 시스템
src/api/services/familyService.ts, src/page/Detail/DetailPage.tsx, src/page/Policy/components/Permisssion.tsx
Permission 인터페이스에 is_enable 필드 추가. 프라이버시 권한 감지를 permissionTitle과 is_enable 모두 확인하도록 업데이트. 권한 UI에서 동일한 userName을 가진 행들에 전화번호 뒷자리(slice(-4)) 추가 표시. familyService.getMembersSimple() 쿼리 추가.
폴링 간격 조정
src/page/Main/MainPage.tsx, src/page/Detail/DetailPage.tsx
네 개의 useQuery 설정(familyMembers, UsageData, sharedPool, blockStatus)에서 refetchInterval을 1000ms에서 10000ms로 증가.
헤더 UI 스타일링
src/components/Header.tsx
비홈 헤더 제목 요소에 수직 정렬을 위한 레이아웃 클래스(h-16 flex items-center) 추가. 알람/설정 아이콘 버튼 크기를 w-12 h-12에서 w-11 h-11로 감소. 아이콘 버튼 그룹 간격을 gap-3에서 gap-2로 조정.
차트 표시 로직
src/page/Detail/components/AppUsageChart.tsx
선택적 isOwnData prop 추가(기본값 false). shouldShowData = isOwnData || isPublic 로직 도입. isOwnData가 true인 경우 isPublic 상태와 관계없이 차트 콘텐츠 표시.
데이터 공개 여부 로직
src/page/Detail/DetailPage.tsx
권한 부재 시 forcibly isPublic = true 설정 제거. isPublic은 이제 API 응답(parsed.updatedIsPublic)에서만 파생. AppUsageChart props 업데이트: showToggle = isOwnData && hasPrivacyPermission, isOwnData={isOwnData} prop 추가.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

  • Feature/policy api #44: 권한 타입, 권한 서비스 및 관련 권한 UI 로직을 도입하며 Permission.is_enable 추가 및 사용과 직접 겹침. 또한 MainPage 쿼리 폴링도 수정.
  • Feature/policy api #61: src/page/Main/MainPage.tsx에서 familyMembers refetchInterval을 10000ms로 조정하는 동일한 폴링 간격 수정.
  • design: 헤더 구조 수정 #54: src/components/Header.tsx 헤더 레이아웃/스타일링 변경(헤더 구조, 크기 클래스 및 아이콘/버튼 크기 조정).
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/policy-api

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@haeni82 haeni82 merged commit 84805d3 into develop Mar 23, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant