[Deploy] develop → main 배포 반영#270
Merged
Merged
Conversation
overview.ts, platform.ts에 중복 선언되어 있던 IRoasRanking을 provider.ts 단일 출처로 이동하고 각 파일에서 re-export
IMetricsResponse, IRoasRanking을 common.ts 단일 출처로 이동 IPlatformPerformance는 IMetricsResponse를 extend해 중복 필드 제거 overview.ts, platform.ts는 common.ts에서 re-export
중복된 예산 타입 2개를 common.ts의 IBudgetResponse 단일 타입으로 통합 소비 파일(api, mock) import명 일괄 수정
provider 유무로 query key 분기해 캐시 네임스페이스 유지 중복된 상수(WARNING/DANGER_THRESHOLD)와 select 로직 단일화
useOverviewMetrics의 toKpis, SinglePlatformView의 인라인 변환 제거 타이틀 문자열과 소수점(2자리) 통일
PlatformRoasTable, TopPerformanceList의 중복 로고 선언 제거 각 컴포넌트는 공통 상수에서 컴포넌트 레퍼런스를 가져와 고유 className 적용
TrafficChart, PlatformTrafficChart의 YYYYMMDDHHmm → timestamp 변환 중복 제거
…ormRoasRankings 날짜 상수화 AllPlatformTrafficChart의 인라인 파싱 로직을 parseMinuteToTimestamp 유틸로 교체 usePlatformRoasRankings의 하드코딩 날짜를 OVERVIEW_DAILY_METRICS_RANGE 상수로 교체
useCoreMutation 내부와 useAiAnalysisReport에서 모듈 레벨 싱글턴을 직접 import하던 방식을 useQueryClient() 훅으로 변경. 테스트 환경에서 QueryClientProvider 주입 시 동일 인스턴스를 참조하도록 수정.
[Refactor/#240] queryClient 싱글턴 import를 useQueryClient() 훅으로 교체
axiosInstance 인터셉터가 401 재발급을 별도로 처리하므로 전역 retry: 1이 4xx 에러에서 불필요한 중복 재시도를 유발하는 문제를 수정
백그라운드 갱신 의도를 void로 명시해 암묵적 Promise 무시 패턴 제거
setIsOpen(false)가 await Promise.all 이전에 실행되어 드롭다운이 캐시 갱신 전에 닫히던 문제를 수정
Workspace.tsx: 워크스페이스 생성 후 setCreateOpen(false) 이전에 캐시 갱신 완료 보장 MemberManagement.tsx: 멤버 삭제 후 setIsDeleteModalOpen(false) 이전에 캐시 갱신 완료 보장
[Feature/#254] 타임라인 기간 선택 컴포넌트 UI 구현
[Bugfix/#249] 사이드바 아이콘 스타일 통일 및 접기 버튼 호버 크기 조정
[Feature/#244] 플랫폼 연동 목록 및 연동 시작 API 연동
[Feature/#252] 타임라인 본체 UI 컴포넌트 및 Storybook 구현
[Feature/#257] 타임라인 성과 요약 패널 UI 구현
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (7)
📒 Files selected for processing (106)
📝 WalkthroughWalkthrough
Changes대시보드 지표 표준화 및 쿼리 인프라
플랫폼 연동 기능
타임라인 기능 초기 구현
Sequence Diagram(s)sequenceDiagram
participant 사용자
participant PlatformIntegrationsPage
participant usePlatformConnections
participant 백엔드
participant 브라우저
rect rgba(59, 130, 246, 0.5)
note over PlatformIntegrationsPage, 백엔드: 연동 목록 조회
PlatformIntegrationsPage->>usePlatformConnections: mount
usePlatformConnections->>백엔드: GET /api/platform/{orgId}/accounts
백엔드-->>usePlatformConnections: IPlatformAccountsResponseData
usePlatformConnections-->>PlatformIntegrationsPage: IPlatformConnectionItem[] (connected/disconnected/error)
end
rect rgba(16, 185, 129, 0.5)
note over 사용자, 브라우저: Google/Meta OAuth 연동
사용자->>PlatformIntegrationsPage: 연동하기 클릭 (GOOGLE)
PlatformIntegrationsPage->>백엔드: GET /api/google/login (Accept: application/json)
백엔드-->>PlatformIntegrationsPage: {redirectUrl}
PlatformIntegrationsPage->>브라우저: window.location.assign(redirectUrl)
end
rect rgba(245, 158, 11, 0.5)
note over 사용자, 백엔드: Naver 자격증명 연동
사용자->>PlatformIntegrationsPage: 연동하기 클릭 (NAVER)
PlatformIntegrationsPage->>사용자: NaverConnectModal 표시
사용자->>PlatformIntegrationsPage: customerId/apiKey/secretKey 입력 후 제출
PlatformIntegrationsPage->>백엔드: POST /api/platform/{orgId}/accounts/naver
백엔드-->>PlatformIntegrationsPage: 성공 응답
PlatformIntegrationsPage->>usePlatformConnections: platform-connections 쿼리 무효화
end
Estimated code review effort🎯 5 (Critical) | ⏱️ ~120 minutes Possibly related PRs
Suggested reviewers
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🚨 관련 이슈
N/A
✨ 변경사항
✏️ 작업 내용
N/A
😅 미완성 작업
N/A
📢 논의 사항 및 참고 사항
N/A
Summary by CodeRabbit
New Features
Bug Fixes