Skip to content

[CHA-77] fix: 저장된 결과 더보기 조회 API 연결 - #261

Merged
yereong merged 4 commits into
mainfrom
cha-77-fix-saved-results-api
Aug 21, 2026
Merged

[CHA-77] fix: 저장된 결과 더보기 조회 API 연결#261
yereong merged 4 commits into
mainfrom
cha-77-fix-saved-results-api

Conversation

@yereong

@yereong yereong commented Aug 21, 2026

Copy link
Copy Markdown
Member

무엇을 변경했나요?

  • /mypage/saved-results 화면에서 저장된 채널 추천·비교·시뮬레이션 조회 API를 연결했습니다.
  • 탭별 페이지 번호와 페이지네이션을 API 요청에 반영했습니다.
  • API 응답을 기존 카드 UI 모델로 매핑하고 로딩·에러·빈 상태를 유지했습니다.
  • 추천 채널 상세 헤더가 추천 응답의 wordmarkUrl을 사용하도록 매핑되는지 검증하고, 값이 없을 때 기존 fallback 이미지를 사용하도록 테스트했습니다.
  • 채널 비교 상세의 API iconUrl 및 추천 채널 이미지가 cdn.chaeso.zip에서 로드될 수 있도록 Next Image 원격 이미지 도메인을 허용했습니다.
  • 탭 전환, 페이지 이동, API 응답 매핑, 채널 이미지 fallback을 검증하는 테스트를 추가·업데이트했습니다.
  • 로그아웃 후 라우팅을 홈 랜딩화면으로 수정하였습니다.

왜 변경했나요?

기존 저장된 결과 더보기 화면이 실제 조회 API와 연결되지 않아 사용자의 저장 결과를 정상적으로 확인할 수 없었습니다. 또한 채널 추천·비교 상세에서 API가 제공하는 채널 이미지가 원격 이미지 도메인 설정으로 차단될 수 있어 화면에 정상적으로 표시되도록 보완했습니다.

Linear: https://linear.app/chaesozip/issue/CHA-77/feature-저장된-결과-탭-및-목록

Closes CHA-77

확인 사항

필수 확인

  • node --run build 통과 — Next 최적화 단계에서 약 2분간 추가 출력 없이 진행되어 중단했습니다.
  • node --run lint 통과 — 기존 tests/design-tokens.test.js의 상대 경로 import 경고 3건이 있습니다.
  • 관련 테스트 통과 — node --run test:ci -- src/pages/recommend-result/model/recommended-channels.test.ts src/pages/compare/model/channel-comparison-adapter.test.ts src/pages/compare/ui/compare-result-channel-cards.test.tsx (3 files, 23 tests)
  • 관련 이슈를 연결했습니다. Closes CHA-77

기능 검증

  • 변경된 기능을 로컬에서 직접 확인했습니다
  • 사용자 플로우가 의도대로 동작하는지 관련 테스트로 확인했습니다
  • API 연동 또는 상태 관리 변경 사항을 확인했습니다
  • 에러/로딩/빈 상태 등 예외 케이스를 확인했습니다
  • 변경된 동작에 대한 테스트를 추가하거나 업데이트했습니다

설정 및 문서

  • 사용자에게 노출되는 문서 변경 사항이 없습니다
  • 새로운 의존성이나 설정 변경 사항을 PR 설명에 작성했습니다
  • 환경 변수 추가/변경 사항이 없습니다
  • 민감한 값, 토큰, 인증 정보가 포함되지 않았습니다

해당되는 경우

  • 보안 영향이 없습니다
  • 성능 영향을 검토했습니다
  • 기존 동작을 변경하는 내용을 PR 설명에 작성했습니다
  • 스크린샷 또는 화면 녹화를 첨부했습니다

@linear-code

linear-code Bot commented Aug 21, 2026

Copy link
Copy Markdown

CHA-77

@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 66765e63-0b02-4aa4-b772-b22d405d83d2


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.

@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.00000% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/app/layouts/session-page-header.tsx 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yereong
yereong marked this pull request as ready for review August 21, 2026 06:34
@github-actions
github-actions Bot requested a review from DongjaJ August 21, 2026 06:35
Comment on lines +29 to +42
const comparisonsQuery = useQuery({
...getMyChannelComparisonsOptions({
query: { page: pages.comparison - 1, size: SAVED_RESULTS_PAGE_SIZE },
}),
enabled,
retry: false,
});
const simulationsQuery = useQuery({
...getMySimulationsOptions({
query: { page: pages.simulation - 1, size: SAVED_RESULTS_PAGE_SIZE },
}),
enabled,
retry: false,
});

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

여긴 useQueries를 써도 될 것 같아요

@yereong
yereong merged commit 0fd3448 into main Aug 21, 2026
7 checks passed
@yereong
yereong deleted the cha-77-fix-saved-results-api branch August 21, 2026 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants