Skip to content

Comments

fix: #793 질문 추가 기능 개선 및 버튼 비활성화 로직 추가#794

Merged
prgmr99 merged 2 commits intodevelopfrom
793-템플릿-질문-추가-직접작성-탭란-버튼-비활성화-제공-필요-및-추천질문-선택과-함께-작성해-추가-시-해당-질문들-모두-질문-추가-되도록-수정합니다
Feb 1, 2026

Hidden character warning

The head ref may contain hidden characters: "793-\ud15c\ud50c\ub9bf-\uc9c8\ubb38-\ucd94\uac00-\uc9c1\uc811\uc791\uc131-\ud0ed\ub780-\ubc84\ud2bc-\ube44\ud65c\uc131\ud654-\uc81c\uacf5-\ud544\uc694-\ubc0f-\ucd94\ucc9c\uc9c8\ubb38-\uc120\ud0dd\uacfc-\ud568\uaed8-\uc791\uc131\ud574-\ucd94\uac00-\uc2dc-\ud574\ub2f9-\uc9c8\ubb38\ub4e4-\ubaa8\ub450-\uc9c8\ubb38-\ucd94\uac00-\ub418\ub3c4\ub85d-\uc218\uc815\ud569\ub2c8\ub2e4"
Merged

fix: #793 질문 추가 기능 개선 및 버튼 비활성화 로직 추가#794
prgmr99 merged 2 commits intodevelopfrom
793-템플릿-질문-추가-직접작성-탭란-버튼-비활성화-제공-필요-및-추천질문-선택과-함께-작성해-추가-시-해당-질문들-모두-질문-추가-되도록-수정합니다

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
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch 793-템플릿-질문-추가-직접작성-탭란-버튼-비활성화-제공-필요-및-추천질문-선택과-함께-작성해-추가-시-해당-질문들-모두-질문-추가-되도록-수정합니다

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에 대한 분기로직이 뭔가 안에 구조체가 달라서 들어간 것인지가 궁금해요! (아니면 행위 별로 구분을 한 것인지?!)

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

Copy link
Member Author

Choose a reason for hiding this comment

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

아 이게, 제가 리뷰를 제대로 못하고 반영한 것 같아요..!
확인해보니까 개수로 분기처리를 진행할 필요가 없는 것 같아요.

추가된 개수에 따라 토스트 메세지로 몇개의 질문이 추가되었는지 피드백을 주기위함이었고, 이거는 유지보수 관점에서 추가되는 함수가 분리되어 있어서 좋은 구조는 아닌 것 같아요!

그래서 질문 추가 함수는 하나로 통일하고, 분기처리를 제거했습니다!

@prgmr99 prgmr99 merged commit 5f24819 into develop Feb 1, 2026
2 checks passed
@prgmr99 prgmr99 deleted the 793-템플릿-질문-추가-직접작성-탭란-버튼-비활성화-제공-필요-및-추천질문-선택과-함께-작성해-추가-시-해당-질문들-모두-질문-추가-되도록-수정합니다 branch February 18, 2026 11:33
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.

2 participants