Skip to content

Conversation

@grapefruit13
Copy link
Contributor

@grapefruit13 grapefruit13 commented Mar 23, 2025

유형

  • 기능 구현
  • UI 구현
  • 리팩토링
  • 버그 해결
  • 문서 업데이트
  • 기타( )

작업 내용

  • 기존의 /autoplan /autoplan/create 두 페이지에서 일어나는 과정 하나로 통합

스크린샷

image

리뷰 요구사항

  • 기존 meal header 컴포넌트를 그대로 사용하려면 또 prop들이 추가되는데 이게 너무 애매해서 사용하지 않았습니다. 지금 다시 코드 만져보면서 meal header 괜히 분리했다라는 생각이 드네요. 피그마 디자인 수정으로 레이아웃도 조금씩 계속 달라져서 근시일내로 분리했던 컴포넌트 다시 합치고, 안 쓰이는 프롭이나 코드 없는지 다시 점검해봐야할 것 같아요... 이번 pr 올리면서 배운 점이 많네요...... 컴포넌트 필요이상으로 쪼갰다가 전달하는 프롭스만 많아지고 유지보수/디자인 변경하기가 더 힘들어진 것 같아요.

@grapefruit13 grapefruit13 self-assigned this Mar 23, 2025
@grapefruit13 grapefruit13 requested a review from plla2 March 23, 2025 12:52
Copy link
Member

@plla2 plla2 left a comment

Choose a reason for hiding this comment

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

알아가신게 많은 pr이라니.. 이런거 알아갈라고 사이드 돌리는거 아니게씁니까!
고생하셨습니다~!

@grapefruit13 grapefruit13 merged commit b011bf9 into dev Mar 24, 2025
@grapefruit13 grapefruit13 deleted the THKV-149 branch March 24, 2025 05:16
plla2 added a commit to plla2/NNplanner-FE that referenced this pull request Mar 24, 2025
fix: 자동식단생성 - 한 페이지에서 식단제목, 카테고리 변경 및 생성하도록  수정
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.

3 participants