Skip to content

Conversation

@plla2
Copy link
Member

@plla2 plla2 commented Apr 4, 2025

유형

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

작업 내용

  • 설문 질문 추가도 가능하게 변경
  • 설문 응답 top3 날짜를 배열에 3개 보내주도록 수정

리뷰 요구사항

  • 기존에 있던 질문들 수정뿐 아니라 질문 추가도 가능하도록 수정해씁니다

@plla2 plla2 requested a review from grapefruit13 April 4, 2025 04:16
@plla2 plla2 self-assigned this Apr 4, 2025
Copy link
Contributor

@grapefruit13 grapefruit13 left a comment

Choose a reason for hiding this comment

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

오 추가질문 기능 구현하셨군요 고생하셨습니다! 특별한 리뷰는 없고 간단한 코멘트 남겼습니다 승현님이 자율적으로 반영해주시고 머지하시면 될 것 같아여~

{question.answerType === 'text' &&
MANY_INPUT_INDEX.includes(idx) && (
<div className='flex flex-col gap-1'>
{[0, 1, 2].map((subIdx) => (
Copy link
Contributor

Choose a reason for hiding this comment

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

여기 '[0,1,2]'도 MANY_INPUT뭐시기처럼 변수로 분리하면 어떨까 싶어요!

@plla2 plla2 merged commit 76874b5 into dev Apr 5, 2025
1 check passed
@plla2 plla2 deleted the THKV-152 branch April 5, 2025 08:26
plla2 added a commit to plla2/NNplanner-FE that referenced this pull request Apr 5, 2025
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