Skip to content

Conversation

@prgmr99
Copy link
Member

@prgmr99 prgmr99 commented Jan 28, 2026

질문 추가 기능 개선 및 버튼 비활성화 로직 추가


🏄🏼‍♂️‍ Summary (요약)

  • 템플릿 질문 추가 화면(AddQuestionView)의 UX를 개선하여 사용자가 직접 작성한 질문과 추천 질문을 함께 추가할 수 있도록 수정했습니다

🫨 Describe your Change (변경사항)

  • 직접작성 탭에서 입력 없이 버튼 비활성화 기능 추가
  • 직접작성 질문과 추천질문을 동시에 추가할 수 있도록 통합 핸들러 구현
  • 질문 추가 후 자동으로 입력 필드 및 선택 상태 초기화

🧐 Issue number and link (참고)

📚 Reference (참조)

2026-01-28.1.28.20.mov

@prgmr99 prgmr99 self-assigned this Jan 28, 2026
@prgmr99 prgmr99 added the 🫨 bug Something isn't working label Jan 28, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 28, 2026

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • Not Rabbit

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Member

@klmhyeonwoo klmhyeonwoo left a comment

Choose a reason for hiding this comment

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

Good!

questionsToAdd.push(...selectedValues);
}

if (questionsToAdd.length === 1) {
Copy link
Member

@klmhyeonwoo klmhyeonwoo Jan 29, 2026

Choose a reason for hiding this comment

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

갓승준~!

커스텀 질문을 직접 입력한 부분이랑 커스텀 질문을 선택한 부분의 로직이 합쳐지는 부분이

questionToAdd.push( ~ ) 

부분이라고 이해했는데요! 직접 입력을 하고, 또 커스텀 질문을 선택해서 추가하게 되면 직접 입력과 커스텀 질문 모두 추가되는 것 같은데 questionToAdd.length === 1에 대한 분기로직이 뭔가 안에 구조체가 달라서 들어간 것인지가 궁금해요! (아니면 행위 별로 구분을 한 것인지?!)

탭 별로 입력 및 선택 여부 상관없이 모두 추가되는 플로우로 이해를 했는데, 문득 궁금해서 코멘트를 남겨보았습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🫨 bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants