fix: #793 질문 추가 기능 개선 및 버튼 비활성화 로직 추가#794
Hidden character warning
Conversation
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the
✨ Finishing touches🧪 Generate unit tests (beta)
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. Comment |
| questionsToAdd.push(...selectedValues); | ||
| } | ||
|
|
||
| if (questionsToAdd.length === 1) { |
There was a problem hiding this comment.
갓승준~!
커스텀 질문을 직접 입력한 부분이랑 커스텀 질문을 선택한 부분의 로직이 합쳐지는 부분이
questionToAdd.push( ~ )
부분이라고 이해했는데요! 직접 입력을 하고, 또 커스텀 질문을 선택해서 추가하게 되면 직접 입력과 커스텀 질문 모두 추가되는 것 같은데 questionToAdd.length === 1에 대한 분기로직이 뭔가 안에 구조체가 달라서 들어간 것인지가 궁금해요! (아니면 행위 별로 구분을 한 것인지?!)
탭 별로 입력 및 선택 여부 상관없이 모두 추가되는 플로우로 이해를 했는데, 문득 궁금해서 코멘트를 남겨보았습니다!
There was a problem hiding this comment.
아 이게, 제가 리뷰를 제대로 못하고 반영한 것 같아요..!
확인해보니까 개수로 분기처리를 진행할 필요가 없는 것 같아요.
추가된 개수에 따라 토스트 메세지로 몇개의 질문이 추가되었는지 피드백을 주기위함이었고, 이거는 유지보수 관점에서 추가되는 함수가 분리되어 있어서 좋은 구조는 아닌 것 같아요!
그래서 질문 추가 함수는 하나로 통일하고, 분기처리를 제거했습니다!
🏄🏼♂️ Summary (요약)
🫨 Describe your Change (변경사항)
🧐 Issue number and link (참고)
📚 Reference (참조)
2026-01-28.1.28.20.mov