Skip to content

[Refactor]: 상위 목표 뷰 리팩토링#190

Merged
jisooooooooooo merged 9 commits intodevelopfrom
refactor/#186/upperTodo
Sep 19, 2025
Merged

[Refactor]: 상위 목표 뷰 리팩토링#190
jisooooooooooo merged 9 commits intodevelopfrom
refactor/#186/upperTodo

Conversation

@jisooooooooooo
Copy link
Copy Markdown
Collaborator

💡 Summary

close #186

✅ Tasks

useMandalartId 훅 추가

  • 기존에 1로 고정해뒀던 부분을 훅으로 분리해 동적으로 관리할 수 있도록 수정했습니다.

UpperTodo 리팩토링

  • 불필요한 userName prop을 제거했습니다.
  • AI 추천 관련 부분을 useUpperTodoAI 훅으로 분리했습니다.
  • 유틸 함수와 상수들을 별도 파일로 분리했습니다.

👀 To Reviewer

  • 리팩토링하실 때 mandalartId 잘 적용되는지 확인 부탁드려요!
  • 현재는 상위 목표 작성 시 엔터 입력 → 생성 / 수정 → 엔터 입력 → 업데이트 / 삭제 후 엔터 입력 → 삭제 요청하는 흐름인데 사용자 입장에서 직관적이지 않을 수 있다는 생각이 들어서 더 나은 방안을 찾아봐야 할 것 같아요,,
  • 툴팁 위치 고정하는 방식을 좀 더 깔끔하게 바꾸고 싶었는데 방법을 못 찾아서.. 좋은 의견이 있다면 공유 부탁드려욤 🙏🏻

📸 Screenshot

@jisooooooooooo jisooooooooooo self-assigned this Sep 15, 2025
@jisooooooooooo jisooooooooooo linked an issue Sep 15, 2025 that may be closed by this pull request
1 task
@vercel
Copy link
Copy Markdown

vercel bot commented Sep 15, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
ninedot Ready Ready Preview Comment Sep 19, 2025 8:09am

@github-actions
Copy link
Copy Markdown

빌드 결과

빌드 성공 🎉

@github-actions
Copy link
Copy Markdown

🎨 스토리북 배포 완료!

👉 스토리북 미리보기

변경된 컴포넌트의 디자인을 확인해주세요.

Copy link
Copy Markdown
Contributor

@kwonsaebom kwonsaebom left a comment

Choose a reason for hiding this comment

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

수고 너무 너무 많으셨습니다 !! 코드도 너무 깔끔하네요 🤩 👍🏻👍🏻
리뷰 몇 개만 확인해주세요 !

++ useMadalartId 훅 따로 분리해주셔서 여러 파일에서 사용할 수 있을 것 같아요 ! 감사합니다아

@github-actions
Copy link
Copy Markdown

빌드 결과

빌드 성공 🎉

@github-actions
Copy link
Copy Markdown

🎨 스토리북 배포 완료!

👉 스토리북 미리보기

변경된 컴포넌트의 디자인을 확인해주세요.

Copy link
Copy Markdown
Contributor

@shinjigu shinjigu left a comment

Choose a reason for hiding this comment

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

useMandalartId 훅 분리와 상위 목표 뷰에서 불필요하게 props를 넘기던 부분이 정리된 게 넘 좋네요 수고 많으셨습니당 🙊 밑에 리뷰 남겼는데, mandalartId가 0일 때 API 호출을 막는 enabled 가드 같이 얘기해 보면 좋을 것 같아요 !!

@github-actions
Copy link
Copy Markdown

🎨 스토리북 배포 완료!

👉 스토리북 미리보기

변경된 컴포넌트의 디자인을 확인해주세요.

@github-actions
Copy link
Copy Markdown

빌드 결과

빌드 성공 🎉

@jisooooooooooo jisooooooooooo merged commit ef5d781 into develop Sep 19, 2025
6 checks passed
@jisooooooooooo jisooooooooooo deleted the refactor/#186/upperTodo branch September 19, 2025 11:08
@jisooooooooooo jisooooooooooo changed the title Refactor: 상위 목표 뷰 리팩토링 [Refactor]: 상위 목표 뷰 리팩토링 Sep 19, 2025
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.

[Refactor]: 상위 목표 뷰 리팩토링

3 participants