Skip to content

review-8: 8번째 리뷰 요청 - #151

Open
Hyeonjun0527 wants to merge 28 commits into
release/2025-11-01from
release/2025-11-07
Open

review-8: 8번째 리뷰 요청 #151
Hyeonjun0527 wants to merge 28 commits into
release/2025-11-01from
release/2025-11-07

Conversation

@Hyeonjun0527

@Hyeonjun0527 Hyeonjun0527 commented Nov 6, 2025

Copy link
Copy Markdown
Member

PR 설명

서비스 결과물

링크 : https://pull.it.kr/
현재 모든 기능이 구현 완료되었습니다.

이번주 작업 요약

  1. 대시보드 API 제작
  2. 테스트 코드 작성 (wrongAnswer, Folder, QuestionSet, Source, Auth)
  3. spring api docs 작성
  4. 리팩토링

작업 내용

다음주 예정 작업

  • qa를 진행하며 발견된 모든 버그를 잡기 위해 리팩토링

궁금한점 및 리뷰포인트

OUT OF MEMORY 문제를 해결하기위해 래빗엠큐 메세지 큐 도입 후, 문제생성 서버 별도 구축 PUB SUB 방식으로 비동기소통

SSE 통신연결이 불안정한 원인 해결을 위한 리팩토링

각종 테스트 코드 작성

그라파나 프로메테우스 도입

무중단 배포 도입

로컬환경에서 운영환경과 같은 다중애플리케이션을 띄우기 위한 구축

Summary by CodeRabbit

  • Style
    • Improved code formatting in test files for consistency and readability.

flareseek and others added 28 commits November 1, 2025 21:22
* fix: add cache clear

* fix: fix sse, heartbeat

* fix: fix sse (reconnection)

* fix: add question title (notification)
refactor: learnstat response 반영 및 markresult로직 수정
* refactor: learnstat response 반영 및 markresult로직 수정

* refactor: 잘못된 로직 수정 및 총 맞은 문제 수 추가

* refactor: 코드스타일
* feat: 프메 그라파나 적용

* refactor: 명칭변경

* feat: learnstatresponse 수정

* refactor: 이름 변경
* feat: 프메 그라파나 적용

* refactor: 명칭변경

* feat: learnstatresponse 수정

* refactor: 이름 변경

* refactor: 코드스타일
* feat: 프메 그라파나 적용

* refactor: 명칭변경

* feat: learnstatresponse 수정

* refactor: 이름 변경

* refactor: 코드스타일

* refactor: 경로 및 체크스타일

* refactor: 코드스타이

* refactor: 코드스타일 재수정

* refacot

* bts

* btsk:wq
* feat: 레디스로 refresh-token 관리

* refactor: 코드스타일

* fix: test code fix

* refactor: 안쓰는 import 제거 및 경고 제거

* fix:코드스타일변경

* fix: 로그보기

* fix: ci.yml

* fix: 버그수정

* fix: 코드스타일
* feat: testcode and api docs

* Merge remote-tracking branch 'origin/develop' into xqqldir/btsk-103/questionset-testcode

* refoactor: checksytle 수정

* refactor

---------

Co-authored-by: Hyeonjun0527 <osumaniaddict527@gmail.com>
* feat: 문제집 생성중 -> 생성실패 스케줄러 작성

* feat: 프로덕션 코드만 수정 테스트 코드 미수정

* feat: 재시도 스케줄링

* feat: 테스트코드 작성

* fix: 충돌 에러

* refactor: 코드스타일 수정

* fix: 테스트 코드 수정

* refactor: 테스트 코드 작성

* refactor: 코드스타일

* 잘됨

* refactor: 코드스타일

* refactor: 코드스타일

* refactor: 코드스타일

* refactor: 코드스타일
* feat: 테스트 코드 작성

* refactor: 코드스타일
* fix: 카카오로 리다이렉트하는 무한루프 로직 수정

* refactor: 코드스타일 수정
* feat: 래빗엠큐 도입

* feat: 래빗엠큐

* refactor: s3 저장소 이름 변경

* 코드스타일

* refactor: 리소스 누수 문제 해결

* 코드스타일

* 코드스타일
refactor: config설정 래빗엠큐 아이디
* fix: sse disconnect solve

* fix: 로그 추가 및 쓸모없는 에러로그 삭제 및 빈주입문제 해결
* fix: 낙관적락을 비관적 락으로 변경

* fix: 테스트 코드 수정 및 폴더 삭제에 기본폴더 이전 로직에 생긴 문제 해결

* refactor: 코드스타일

* refactor: 코드스타일

* fix: 분산환경에서 스킵 락 필요

* fix: 테스트 코드 수정

* fix: 어노테이션 적용 네이티브쿼리에 적용안되는 문법 제거

* fix: 폴더없을시 보안로직

* fix: 코드스타일

* fix:코드스타일

* fix: 코드스타일

* fix: 테스트코드
* feat: 로컬 개발 환경 Docker 전환 및 전체 버그 수정

* fix: 에러 해결
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@coderabbitai

coderabbitai Bot commented Nov 6, 2025

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

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.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch release/2025-11-07

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants