Skip to content

fix : 기록 화면 지도 개선 및 통계 하드코딩 (테스트)#128

Merged
peisonger merged 2 commits into
mainfrom
fix/home
Jun 4, 2026
Merged

fix : 기록 화면 지도 개선 및 통계 하드코딩 (테스트)#128
peisonger merged 2 commits into
mainfrom
fix/home

Conversation

@peisonger

Copy link
Copy Markdown
Contributor

Summary

  • GPS 트랙 없을 때 코스 폴리라인으로 NaverMapView 표시
  • GPS 트랙 있을 때 노란 경로 + 파란/빨간 시작·끝 점 마커
  • 사진 있을 때 원형 사진 마커 추가
  • 기록 화면 통계(소요시간/고도/칼로리) 테스트용 하드코딩

Test plan

  • 기록 화면 진입 시 NaverMapView 정상 렌더링 확인
  • GPS 트랙 없는 기록 → 코스 폴리라인 표시 확인
  • 통계 하드코딩 값(3시간 24분 / 642m / 1128kcal) 확인

⚠️ 통계 하드코딩은 테스트용이므로 API 연동 후 되돌려야 합니다.

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request replaces dynamic statistical data (duration, altitude, and calories) with hardcoded test values in the record screen. The reviewer pointed out that these hardcoded values should be reverted to dynamic data for production and also identified a typo in the original code where the altitude unit was incorrectly labeled as 'Nm' instead of 'm', providing a code suggestion to resolve both issues.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread app/record/[id].tsx
Comment on lines +275 to +277
{ label: "소요시간", value: "3시간 24분" },
{ label: "고도", value: "642m" },
{ label: "칼로리", value: "1128kcal" },

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

high

테스트를 위해 통계 데이터를 하드코딩한 부분은 실제 서비스에 반영되기 전에 동적 데이터를 사용하도록 되돌려야 합니다.

추가로, 기존 코드에서 고도 단위가 ${Math.round(recordDetail.ascentMeters)}Nm로 표기되어 Nm이라는 오타가 있었습니다. 이를 올바른 단위인 m으로 수정하여 복원하는 것을 제안합니다.

Suggested change
{ label: "소요시간", value: "3시간 24분" },
{ label: "고도", value: "642m" },
{ label: "칼로리", value: "1128kcal" },
{ label: "소요시간", value: formatDuration(recordDetail?.durationSeconds ?? durationSec) },
{ label: "고도", value: recordDetail?.ascentMeters != null ? Math.round(recordDetail.ascentMeters) + "m" : "--" },
{ label: "칼로리", value: recordDetail?.calories != null ? recordDetail.calories + "kcal" : "--" },

@peisonger peisonger merged commit 8874874 into main Jun 4, 2026
2 checks passed
@peisonger peisonger deleted the fix/home branch June 4, 2026 17:57
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