-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature] 스터디 수강생 페이지네이션 적용, 엑셀 다운로드 #160
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고 많으셨습니다~~ 마이너 리뷰라서 미리 어푸룹 할게용
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
정말 수고 많으셨습니다!! 말씀해주신 Table 컴포넌트 관련 이슈 확인해볼게용!! 💨
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Table 관련된 이슈는 코드보면서 디잔시스템에 수정해보도록 하겠습니다!
GDSC-Hongik/wow-design-system#174 |
감사합니당~!! 디쟌시스템 배포되면 다음 PR에 반영하께요 |
chore: 디자인시스템 0.1.19 버전업 feat: 추가 및 변경된 DTO 등록(StudyStudentApiResponseDto, PageStudyStudentApiResponseDto) feat: 페이지네이션 임시적용 feat: 디자인 시스템 테이블 컴포넌트 적용 feat: 페이지네이션 적용 feat: 우수회원 배지 AwardIcon 컴포넌트 feat: 1차 우수회원, 2차 우수회원 표 적용 feat: 수료 체크박스 StarCheckIcon 컴포넌트 feat: 수료여부 표 적용 refactor: 수강생 Th 정보 배열로 관리 feat: 과제 제출 타입 엔티티 및 태그 맵 객체 feat: 과제 제출 태그 표 적용 feat: task, attendance 타입 엔티티 및 studyTasks 타입 변경 feat: 과제 출석 태그 맵 객체 fix: 중복된 key 유니크하게 바꾸기 feat: 과제 출석 태그 적용 refactor: StudentListItem 컴포넌트 분리 feat: 출석률, 과제 수행률 표 적용 fix: 페이지네이션 잘못된 형식 수정 fix: 과제 상태 NOT_SUBMITTED 텍스트 변경 feat: 수강생 필터 UI feat: 엑셀 다운로드 버튼 fix: 수강생 없을 때와 데이터 받는 중일 때 구분 fix: application/octet-stream 타입도 blob으로 응답 읽도록 수정 feat: 수강생 엑셀 다운로드, 스터디리스트 코어멤버도 자신이 생성한 스터디만 보이도록 임시수정 fix: CANCELED로 프로퍼티 이름 변경 fix: 타입 에러 수정 fix: 필터 로직 임시삭제 fix: Tr value값 넘겨주기 refactor: StudentList 테이블 컴포넌트 분리 refactor: 테이블 관련 컴포넌트 폴더 이동 fix: AwardIcon 활성화 시 컬러 미적용 문제 해결 refactor: 타입에 따라 태그 정보를 반환하는 formatTaskToTagInfo 함수 refactor: 불필요한 삼항연산자 개선 refactor: 수강생 페이지네이션 DTO 네이밍 변경
🎉 변경 사항
CANCELED
속성 이름을 변경합니다.🚩 관련 이슈
🙏 여기는 꼭 봐주세요!
Table
컴포넌트에서 문제가 되었던 부분은value
값을 사용하는Tr
이었어서, 해당 부분만 컨텍스트(Table
) 하위에 놓고 나머지를 분리해서 해결했습니다.value
를 사용하지 않는 경우 컨텍스트가 필요하지 않을 것 같아서 (상태관리 없는 단순 표인 경우) 이 부분 이슈로 올려둘게요!