Conversation
student.updated 이벤트 처리 시 DataGSM 내부 PK인 student_id를 학번으로 잘못 사용해 사용자를 찾지 못하고 갱신이 유실되던 문제를 수정합니다. DataGSM 페이로드에는 student_id(내부 PK)와 student_number(학번)가 함께 전달되는데, 기존 코드는 후보 필드를 순서대로 조회해 student_id를 먼저 선택했습니다. 반면 users.student_id 컬럼에는 회원가입 시점부터 학번이 저장되므로(UserRegistrationSupport), 조회가 매번 빗나가 예외 없이 200을 반환한 채 변경 사항이 유실됐습니다. 조회 후보에서 student_id와 studentId를 제거합니다. 폴백으로 남겨두면 동일하게 잘못된 값을 우선 선택하므로 후보에서 완전히 제외했습니다.
기존 테스트는 student_id를 학번으로 가정한 픽스처를 사용해 조회 버그를 그대로 통과시켰습니다. 실제 페이로드 형태에 맞게 픽스처를 정정하고 회귀 테스트를 추가합니다. - student_id(127)와 student_number(3404)가 함께 오는 실제 페이로드로 학번 조회 및 호실 갱신을 검증 - 픽스처의 학번을 실제 형태(3404)로 정정
[datagsm] 학생 이벤트가 학번으로 사용자를 조회하도록 수정
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the DataGSM student event handling logic to extract and identify students using their student number (student_number/studentNumber) instead of the internal DataGSM primary key (student_id/studentId). The corresponding unit tests have been updated to reflect this change, and a new test case has been added to verify correct lookup behavior when both fields are present in the payload. There are no review comments, so I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
3 participants
개요
#99 의 내용을 상용환경에 배포합니다.