You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
가격 포맷팅 처리 (UX/UI)
현재 DeviceSummaryCard.tsx에서 가격을 device?.price 그대로 렌더링하고 있습니다. API 응답 타입에 currency: 'KRW'가 있으므로, 천 단위 콤마와 통화 단위를 붙여주는 것이 사용자 경험상 훨씬 좋을 것 같습니다.
상수 관리 구조 개선
src/constants/lifestyle.ts를 보면 LIFESTYLE_TAGS, LIFESTYLE_TAG_IMAGE_MAP, LIFESTYLE_LABEL_TO_TAGKEY가 서로 다른 객체로 분리되어 있습니다. 이를 Config Object로 묶는 건 어떨까요
useEffect Cleanup 누락 점검
LifestylePage.tsx의 handleClickTag 함수 내에서 setTimeout을 사용하고 있는데, 만약 타임아웃이 돌기 전에 컴포넌트가 언마운트되면 메모리 누수나 상태 업데이트 에러가 발생할 수 있습니다. 이미 useEffect에서 resumeTimerRef를 정리하고 계시지만, 로직 안전성을 위해 handleClickTag 실행 시 기존 타이머를 확실히 clear 해주는 것도 고려해주세요
가격 포맷팅 처리 (UX/UI)
현재 DeviceSummaryCard.tsx에서 가격을 device?.price 그대로 렌더링하고 있습니다. API 응답 타입에 currency: 'KRW'가 있으므로, 천 단위 콤마와 통화 단위를 붙여주는 것이 사용자 경험상 훨씬 좋을 것 같습니다.
상수 관리 구조 개선
src/constants/lifestyle.ts를 보면 LIFESTYLE_TAGS, LIFESTYLE_TAG_IMAGE_MAP, LIFESTYLE_LABEL_TO_TAGKEY가 서로 다른 객체로 분리되어 있습니다. 이를 Config Object로 묶는 건 어떨까요
useEffect Cleanup 누락 점검
LifestylePage.tsx의 handleClickTag 함수 내에서 setTimeout을 사용하고 있는데, 만약 타임아웃이 돌기 전에 컴포넌트가 언마운트되면 메모리 누수나 상태 업데이트 에러가 발생할 수 있습니다. 이미 useEffect에서 resumeTimerRef를 정리하고 계시지만, 로직 안전성을 위해 handleClickTag 실행 시 기존 타이머를 확실히 clear 해주는 것도 고려해주세요
1,2,3 모두 반영했습니다. 감사합니다.
현재는 모든 태그의 응답에서 currency가 KRW로 오는데 추후 KRW가 아닌 다른 통화 단위가 응답으로 들어올 수 있는지 백엔드 측에 문의 드렸습니다. 만약 예외가 있다면 그것까지 수정해서 merge 하겠습니다.
가격 포맷팅 처리 (UX/UI)
현재 DeviceSummaryCard.tsx에서 가격을 device?.price 그대로 렌더링하고 있습니다. API 응답 타입에 currency: 'KRW'가 있으므로, 천 단위 콤마와 통화 단위를 붙여주는 것이 사용자 경험상 훨씬 좋을 것 같습니다.
상수 관리 구조 개선
src/constants/lifestyle.ts를 보면 LIFESTYLE_TAGS, LIFESTYLE_TAG_IMAGE_MAP, LIFESTYLE_LABEL_TO_TAGKEY가 서로 다른 객체로 분리되어 있습니다. 이를 Config Object로 묶는 건 어떨까요
useEffect Cleanup 누락 점검
LifestylePage.tsx의 handleClickTag 함수 내에서 setTimeout을 사용하고 있는데, 만약 타임아웃이 돌기 전에 컴포넌트가 언마운트되면 메모리 누수나 상태 업데이트 에러가 발생할 수 있습니다. 이미 useEffect에서 resumeTimerRef를 정리하고 계시지만, 로직 안전성을 위해 handleClickTag 실행 시 기존 타이머를 확실히 clear 해주는 것도 고려해주세요
1,2,3 모두 반영했습니다. 감사합니다. 현재는 모든 태그의 응답에서 currency가 KRW로 오는데 추후 KRW가 아닌 다른 통화 단위가 응답으로 들어올 수 있는지 백엔드 측에 문의 드렸습니다. 만약 예외가 있다면 그것까지 수정해서 merge 하겠습니다.
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
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.
📌 관련 이슈번호
close : #108
🔍 구현한 내용
라이프스타일 페이지 API 연동했습니다.
tagKey기준으로 대표 디바이스 3개를 조회하고, 하단 기기 요약 카드에 노출되도록 했습니다.라이프스타일 관련 상수와 타입을 분리했습니다.
constants/lifestyle.ts로 이동했습니다.DeviceSummaryCard에 API 응답 데이터를 연결했습니다.
📸 스크린샷 or 실행 영상
2026-02-04.230228.mov
📢 리뷰어에게
tagKey = tour일 때 첫 번째 기기 사진이 제대로 안 보여 pm님께 이미지 url 수정 요청 드린 상황입니다. 연동 상의 이상은 없습니다.