Conversation
shinjigu
requested review from
Leeyoonji23,
jisooooooooooo and
kwonsaebom
as code owners
Collaborator
jisooooooooooo
left a comment
jisooooooooooo
left a comment
There was a problem hiding this comment.
늦게까지 고생하셨습니다 ~~~🤩
정리하느라 힘드셨을 텐데 많이 깔꼼해진 거 같네요
코멘트 확인해 주시면 감사하겠습니당 !👍🏻🐻❄️
| openModal(aiModalContent); | ||
| } catch (error) { | ||
| const message = getServerMessage(error, ALERT.aiFetchFail); | ||
| const failModalContent = <AiFailModal onClose={closeModal} message={message} />; |
Collaborator
There was a problem hiding this comment.
p3) 실패 모달 띄울 때 저도 이 부분 고민하다가 피그마에 '다시 한 번 시도해주세요' 멘트밖에 없길래 그냥 하나로 통일해서 띄우긴 했는데..
개인적으로는 이미 추천 기능을 사용했다는 멘트는 띄워줘도 좋지 않나 생각하긴 합니다 ㅎㅎ
이 부분은 기획 측과 논의 후에 상위, 하위 통일시키면 좋을 거 같네욤
Contributor
Author
There was a problem hiding this comment.
@jisooooooooooo 내일 전체 회의 때 같이 이야기 나눠보고 정해요 !!
Leeyoonji23
requested changes
Sep 22, 2025
Contributor
Leeyoonji23
left a comment
Leeyoonji23
left a comment
There was a problem hiding this comment.
확실히 리팩토링 하니 깔꼼해지긴 했네요 ㅎㅎ 굳굳
코멘트 몇 개 확인해주시면 감사하겠습니다. 고생하셨습니다~!
Leeyoonji23
approved these changes
Sep 22, 2025
Contributor
Leeyoonji23
left a comment
Leeyoonji23
left a comment
There was a problem hiding this comment.
깔꼼스딱스하게 반영 잘 해주셨네요 머지고~
jisooooooooooo
approved these changes
Sep 22, 2025
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.
💡 Summary
하위 목표 뷰는 상위 목표 뷰와 달리 주기 개념이 추가되어 있다는 차이점이 있지만, 상위 목표 뷰와 최대한 동일한 핵심 구조와 기능을 가지도록 전면 리팩토링을 진행했습니다 .. !
기존 lowerTodo의 코드 구조가 upperTodo와 일관성이 없어 개발 효율성이 떨어지는 문제가 있었습니다. 특히 코드 구조 불일치, 중복 코드, 불필요한 주석과 콘솔로그 등으로 인해 유지보수가 어려웠습니다. 그래서 상위와 하위 목표 뷰의 핵심 구조를 통일하여 개발 효율성을 향상시키고, 기존 기능을 유지하면서 코드를 개선하는 것을 목표로 리팩토링을 진행했습니다 !!
✅ Tasks
src/api/domain/lowerTodo/전체 구조를 상위와 일관되게 정리하였고, 불필요한 파일들은 제거했습니다.useLowerTodoState.ts에서 상위와 동일한 패턴으로 CRUD 로직들을 정리했고,useLowerTodoAI.tsx에서 AI 추천 로직을 상위와 일관되게 개선했습니다.LowerTodoHeader.tsx,MandalCompleteButton.tsx를 추가하여 상위와 동일한 구조로 변경했습니다.TodoFields.tsx에서는 할 일 입력 필드들과 작은 Sub 만다라트를 렌더링하는 컴포넌트로, Square 컴포넌트 렌더링 로직을 개선했습니다.LowerTodo.tsx에서 컴포넌트 구조 및 로직을 상위와 일관되게 정리했고,LowerTodo.css.ts에서는 하위 목표 뷰에 대한 모든 스타일들을 정리하고, 지수님이 추가해 주신 레이아웃 토큰 적용 완료했습니다.👀 To Reviewer
하위 목표 뷰 기능이 모두 정상 작동하는지 확인해 주시면 좋을 것 같습니다 ~
📸 Screenshot
아래는 리팩토링된 하위 목표 뷰의 파일 구조입니다.
