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

[Feature] Manager 문제 관리 기능 추가 #18

Merged
merged 16 commits into from
Apr 9, 2024
Merged

[Feature] Manager 문제 관리 기능 추가 #18

merged 16 commits into from
Apr 9, 2024

Conversation

Sosohy
Copy link
Collaborator

@Sosohy Sosohy commented Apr 9, 2024

PR 타입(하나 이상의 PR 타입을 선택해주세요)

  • 기능 추가
  • 기능 삭제
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트

반영 브랜치

manager -> develop

변경 사항

  • 날짜별 크롤링 문제 조회
  • 크롤링 문제 상세 조회
  • 출제 문제 조회
  • 출제할 문제 추가
  • 출제 문제 리스트에서 문제 삭제

테스트 결과

  • 날짜별 크롤링 문제 조회 테스트 및 예외 처리 테스트 통과
  • 크롤링 문제 상세 조회 및 예외처리 테스트 통과
  • 출제 문제 리스트 조회 및 예외처리 테스트 통과
  • 출제할 문제 추가 테스트 통과
  • 출제 문제 리스트에서 문제 삭제 테스트 통과

Sosohy added 15 commits April 4, 2024 12:38
선택 날짜로 일치하는 문제 리스트 조회 및 조회 테스트(4/2 날짜로 조회, 추후 ParameterizedTest로 변경)
선택된 문제 정보 조회 및 조회 테스트(추후 ParameterizedTest로 변경)
출제 선택된 문제 정보 Quiz 객체로 변환 후 DB에 저장 및 테스트
(테스트는 추후 ParameterizedTest로 변경)
출제될 날짜 당일->다음날로 변경
날짜에 따라 출제할 문제 Quiz테이블에서 조회
출제될 문제 리스트에서 출제될 문제 삭제
출제 문제를 저장하는 경우 max 인덱스 값 찾아서 수정도 함께 가능하도록 수정
manage 기능 관련 컨트롤러 추가
테스트 코드에 ParameterizedTest 추가
예외처리 코드 수정 및 테스트 코드 추가
크롤링 문제 확인 시, 화면에서 확인할 수 있도록 변수 및 기능 추가
프론트에서 효율적으로 사용하기 위해 문제 자체를 보내는 것이 아니라 id 하나씩 추가하는 방식으로 변경
메소드 수정에 따라 postMapping을 getMapping으로 변경
originId 받아서 삭제하는 방식으로 deleteQuiz함수 수정
@Leegiyeon Leegiyeon added the enhancement New feature or request label Apr 9, 2024
@Leegiyeon
Copy link
Member

관리자 기반 구현 내용 확인되었습니다.
고생많으셨습니다 :)

@Leegiyeon Leegiyeon merged commit 2315c1b into develop Apr 9, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants