[Feature] 우수회원/수료회원 처리 및 철회 모달 구현

[hamo-o]

🎉 변경 사항

우수/수료 회원 처리, 철회 모달을 구현합니다.

🚩 관련 이슈

• [Feature] 우수회원 처리 및 철회 #153
• [Feature] 수료 회원 처리 및 철회 #165

🙏 여기는 꼭 봐주세요!

• 선택된 수강생 목록 중 가장 첫 수강생의 이름을 가져오는 방법을 고민중입니다. studentList에서 선택된 수강생을 찾도록 find 메서드를 활용하고 있는데, 더 효율적인 방법이 있다면 알려주세요!
• 수료 회원 처리 및 철회도 유사한 로직이라 이번 PR에 포함합니다. 분기처리가 너무 복잡한 것 같아 좋은 방법이 있다면 알려주세요!

[eugene028]

수고하셨어요!
철회 + 우수처리를 한 모달에서 그 기능을 담당하려다 보니 분기 로직이 조금 복잡해진 부분이 있는 것 같아요.
디자인은 유사하지만, 철회, 우수처리를 할 때 각각 다른 모달로 그 기능을 분리하고 해당 모달로 라우트되는 방식으로 리팩토링한다면 복잡한 분기처리를 하지 않아도 될 것 같아요.

ex) handleStudentCompletionModal / updateStudentAchievementModal 모달로 분리

[eugene028]

이미 우수처리를 한 회원을 중복해서 우수처리를 하려고 하면 500에러가 발생해요.
서버에서 수정해야 할 부분인 것 같아요 ㅠ,.ㅠ

[SeieunYoo]

수고 많으셨습니당~~

• 테이블의 인원을 전부 체크해도 헤더의 체크 상태가 활성화 되지 않는 거 같아요! 요 부분도 확인 한번 부탁드려용

[hamo-o]

이미 우수처리를 한 회원을 중복해서 우수처리를 하려고 하면 500에러가 발생해요. 서버에서 수정해야 할 부분인 것 같아요 ㅠ,.ㅠ

피그마 디자인을 확인해 보니, 중복처리의 경우 프론트단에서부터 체크박스를 비활성화 시키고 있어 반영했습니다! 9b805b5
저는 전체선택 체크박스 상태관리가 잘 안되어서 커스텀으로 체크박스를 넣어서 구현이 가능했는데,
요렇게 특정 row 체크박스를 비활성화 시키려면 디자인 시스템 쪽에서도 체크박스 상태를 커스텀할 수 있어야 할 것 같네요!

백엔드분들께도 중복요청 발생 시 에러 발생 또는 정상 응답으로 통일 부탁드렸습니다.

[eugene028]

체크박스 disabled 처리를 Table 컴포넌트 자체로 하지 못하기 때문에, 자체적으로 Checkbox 구현해서 넣어야 하는 문제가 있네요..! 🥲
너무 고생많으셨습니다..!

그리고 확인해보니, 현영님께서 직접 구현해주신 onChange 함수를 활용한다면 와우클래스단에서 학생 배열을 굳이 집합 자료구조를 사용하지 않아도 동작이 잘 되는 것으로 확인되어요! <- 요건 굳이 바꾸지 않으셔도 되나, 기존에 배열로 코드 작성하셨길래 혹시나 언급합니당

[SeieunYoo]

컴포넌트 커스텀 하시느라 너무 수고 많으셨습니다 😵👍 말씀 주신 테이블 이슈는 디자인 시스템 쪽에서 지원하도록 수정되면 좋을 거 같아요
머지고고~