Conversation
* fix: rag 생성 실패 시 fallback 코드 추가 - rag 생성 호출이 실패하면 상태를 failed로 바꾼다. * fix: AI 서버 스펙 변경에 맞춰 API 스펙 수정 및 엔티티 마이그레이션 - 누락된 failed enum을 추가 - 첨삭 총평 필드를 correction_item에서 제거하고 portfolio-correction에 추가 * feat: correctionStatus enum 상태 다양화 - rag가 실패한 경우와 첨삭이 실패한 경우를 분리하기 위해 enum 상태를 추가함. * refactor: try/catch 구문을 사용해 오류 처리 로직 개선 - async/await 비동기 오류 처리로 오류 처리 로직을 더 명확하고 읽기 쉽게 개선
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 포트폴리오 첨삭 시스템의 데이터 구조, 오류 처리 및 데이터 무결성을 개선하는 데 중점을 둡니다. 첨삭 총평의 관리 주체를 변경하고, 포트폴리오 필드의 문자열 길이를 확장하여 더 많은 내용을 담을 수 있도록 했습니다. 또한, AI 첨삭 과정의 견고성을 높이기 위해 새로운 실패 상태를 도입하고, 엔티티 업데이트 시 타임스탬프 관리를 자동화했습니다. 전반적으로 시스템의 안정성과 유연성을 향상시키는 변경사항입니다. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
이 PR은 첨삭 결과 저장 로직을 리팩토링하고, AI 서버 연동 시 오류 처리 로직을 강화하며, 여러 엔티티의 updatedAt 필드가 누락되던 문제를 수정하는 등 다수의 개선 사항을 포함하고 있습니다. 특히 overallReview를 CorrectionItem에서 PortfolioCorrection으로 이동시킨 점과, 비동기 작업 실패 시 상태를 FAILED 또는 RAG_FAILED로 전이시키는 로직 추가는 시스템의 안정성을 크게 향상시킬 것입니다. 전반적으로 코드 품질을 높이는 좋은 변경 사항들이며, 한 가지 타입 단언과 관련된 작은 개선점을 코멘트로 남겼습니다.
src/modules/internal/presentation/internal-correction-result.controller.ts
Show resolved
Hide resolved
- 채팅 멘션 주석처리
* refactor: Admin 이벤트 보상 시스템 단순화 및 레거시 정리 (#319) - Notice 수동 입력 필드 제거, event DB 값 기반 자동 구성 - CS 이벤트 다중 지급 지원 (allowMultipleRewards) - CS 커스텀 이용권 선택 (customRewards DTO 추가) - 어드민 이벤트 드롭다운 기간 필터 제거 - CTA 자동 매핑 (보상 종류 기반) - POST /admin/api/tickets/grant API 제거 (CS 이벤트로 통합) - GET /admin/api/tickets/history API 제거 (ticket-grants로 통합) - Admin UI: 이용권 거래 내역 탭/이용권 지급 버튼 제거 - Admin UI: GrantEventRewardModal 리팩터링 - manual-reward-options에 userId 쿼리 추가 (isGranted 포함) - FEEDBACK/CS 이벤트 seed 마이그레이션 추가 * docs: API.md 제거된 엔드포인트 반영 (#319) - POST /admin/api/tickets/grant 제거 반영 - GET /admin/api/tickets/history 제거 반영 - GET /admin/api/events/manual-reward-options 문서 추가 * refactor: createNotice 체크박스 제거, 항상 notice 생성 (#319) - Admin UI: 보상 안내 모달 체크박스 제거 → 항상 생성 - AdminGrantRewardReqDTO: createNotice 필드 제거 - buildNoticeFromEvent: createNotice 파라미터 제거 * fix: 이벤트 옵션 로드 실패 시 에러 메시지 표시 (#319)
* Fix: 포트폴리오 내용 글자수 임시 변경 (#313) * Fix: rag 요청 실패 시 fallback 로직 추가 (#312) * fix: rag 생성 실패 시 fallback 코드 추가 - rag 생성 호출이 실패하면 상태를 failed로 바꾼다. * fix: AI 서버 스펙 변경에 맞춰 API 스펙 수정 및 엔티티 마이그레이션 - 누락된 failed enum을 추가 - 첨삭 총평 필드를 correction_item에서 제거하고 portfolio-correction에 추가 * feat: correctionStatus enum 상태 다양화 - rag가 실패한 경우와 첨삭이 실패한 경우를 분리하기 위해 enum 상태를 추가함. * refactor: try/catch 구문을 사용해 오류 처리 로직 개선 - async/await 비동기 오류 처리로 오류 처리 로직을 더 명확하고 읽기 쉽게 개선 * fix: createdAt 시간 오류 해결 (#315) * Fix: 인사이트로그 시간 에러 해결 (#317) - 채팅 멘션 주석처리 * Fix: 시간 UTC로 롤백 (#318) * Refactor: Admin 이벤트 보상 시스템 단순화 및 레거시 정리 (#319) (#320) * refactor: Admin 이벤트 보상 시스템 단순화 및 레거시 정리 (#319) - Notice 수동 입력 필드 제거, event DB 값 기반 자동 구성 - CS 이벤트 다중 지급 지원 (allowMultipleRewards) - CS 커스텀 이용권 선택 (customRewards DTO 추가) - 어드민 이벤트 드롭다운 기간 필터 제거 - CTA 자동 매핑 (보상 종류 기반) - POST /admin/api/tickets/grant API 제거 (CS 이벤트로 통합) - GET /admin/api/tickets/history API 제거 (ticket-grants로 통합) - Admin UI: 이용권 거래 내역 탭/이용권 지급 버튼 제거 - Admin UI: GrantEventRewardModal 리팩터링 - manual-reward-options에 userId 쿼리 추가 (isGranted 포함) - FEEDBACK/CS 이벤트 seed 마이그레이션 추가 * docs: API.md 제거된 엔드포인트 반영 (#319) - POST /admin/api/tickets/grant 제거 반영 - GET /admin/api/tickets/history 제거 반영 - GET /admin/api/events/manual-reward-options 문서 추가 * refactor: createNotice 체크박스 제거, 항상 notice 생성 (#319) - Admin UI: 보상 안내 모달 체크박스 제거 → 항상 생성 - AdminGrantRewardReqDTO: createNotice 필드 제거 - buildNoticeFromEvent: createNotice 파라미터 제거 * fix: 이벤트 옵션 로드 실패 시 에러 메시지 표시 (#319) --------- Co-authored-by: 강효인 <hyoin1704@gmail.com>
Summary
OBT 오픈 전 수정사항 반영
Type of Change
해당하는 항목에 체크해주세요:
Target Environment
배포 대상 브랜치를 선택해주세요:
dev)main)