Skip to content
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

[FIX]심사보고서 이름 포맷 수정 & 카프카 반영 #151

Merged
merged 22 commits into from
Oct 18, 2024
Merged

Conversation

jcy0308
Copy link
Member

@jcy0308 jcy0308 commented Oct 18, 2024

작성자: @jcy0308

체크 리스트

  • 적절한 제목으로 수정했나요?
  • 상단에 이슈 번호를 기입했나요?
  • Target Branch를 올바르게 설정했나요?
  • Label을 알맞게 설정했나요?

작업 내역

  • main 브랜치 CI/CD 변경 -> 이제 harbor를 사용하지 않습니다.
  • pdf 생성은 비동기 로직으로 변경& 카프카 이용
  • 심사보고서에 학생이름과 교수님이름이 추가됩니다.

비고

  • 참고 사항을 적어주세요. 코드 리뷰하는 사람이 참고해야 하는 내용을 자유로운 형식으로 적을 수 있습니다.

resolve #100
resolve #148

hynseok and others added 22 commits May 9, 2024 11:24
* fix: 지도 관계 수정 API에서 기존 배정 인원 확인하여 error를 보내는 로직 삭제

* [HOTFIX] 심사의견 양식 수정  /  최종판정 심사의견 확인 가능하도록 수정 (#125)

* fix: 심사보고서 양식 수정

* fix: /reviews/me otherReviews Dto 수정

* fix: 심사현황 조회 심사가 끝난 정보 남기도록 수정 (#131)

* fix: 학생 엑셀 등록 timeout 40초로 수정 (#124)

* fix: 심사논문 관련 추가 수정 (#129)

* fix: 논문 제출이 안된 학생의 심사를 진행할 수 없도록 수정

* fix: 심사위원장이 전원 심사완료가 아닐시 심사할수 없도록 수정

* 추가수정

* fix: filter 조건 수정

* fix: 누락된 query 조건 추가

* fix(be): query typo 수정

* [FIX] 특정 학과 학생 논문 정보 업데이트 오류 수정 (#130)

* fix: 학생이 본심 논문 수정 시 수정지시사항 반영 학과 체크

* fix: 관리자가 본심 논문 수정 시 수정지시사항 반영 학과 체크

* fix: 논문 정보 업데이트 API 다시 수정

* fix: 학생 엑셀 업로드로 논문 제목 수정시 생길 수 있는 오류 수정

---------

Co-authored-by: yesjuhee <juhee0924@g.skku.edu>
Co-authored-by: chanyeong <60565169+jcy0308@users.noreply.github.com>
* fix: 지도 관계 수정 API에서 기존 배정 인원 확인하여 error를 보내는 로직 삭제

* [HOTFIX] 심사의견 양식 수정  /  최종판정 심사의견 확인 가능하도록 수정 (#125)

* fix: 심사보고서 양식 수정

* fix: /reviews/me otherReviews Dto 수정

* fix: 심사현황 조회 심사가 끝난 정보 남기도록 수정 (#131)

* fix: 학생 엑셀 등록 timeout 40초로 수정 (#124)

* fix: 심사논문 관련 추가 수정 (#129)

* fix: 논문 제출이 안된 학생의 심사를 진행할 수 없도록 수정

* fix: 심사위원장이 전원 심사완료가 아닐시 심사할수 없도록 수정

* 추가수정

* fix: filter 조건 수정

* fix: 누락된 query 조건 추가

* fix(be): query typo 수정

* [FIX] 특정 학과 학생 논문 정보 업데이트 오류 수정 (#130)

* fix: 학생이 본심 논문 수정 시 수정지시사항 반영 학과 체크

* fix: 관리자가 본심 논문 수정 시 수정지시사항 반영 학과 체크

* fix: 논문 정보 업데이트 API 다시 수정

* fix: 학생 엑셀 업로드로 논문 제목 수정시 생길 수 있는 오류 수정

* fix: 논문 정보 업데이트 api 파일 업데이트 부분 수정

---------

Co-authored-by: yesjuhee <juhee0924@g.skku.edu>
Co-authored-by: chanyeong <60565169+jcy0308@users.noreply.github.com>
…rmat

[FIX] ISSN 미입력시 미제출 표시에서 빈칸으로 표시하도록 수정
@jcy0308 jcy0308 added the fix label Oct 18, 2024
@jcy0308 jcy0308 self-assigned this Oct 18, 2024
@hynseok hynseok merged commit eccce19 into main Oct 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

결과보고서 파일 이름 수정 🚀 Implement Word/PDF Module
2 participants