Skip to content

fix: 전수조사 버그 수정 (학번 변경, 오늘 날짜, 불러오기 실패 표시) - #113

Open
wlsgur11 wants to merge 2 commits into
developfrom
fix/audit-2026-09-23
Open

wlsgur11 wants to merge 2 commits into
developfrom
fix/audit-2026-09-23

Conversation

@wlsgur11

@wlsgur11 wlsgur11 commented Sep 23, 2026 •

Copy link
Copy Markdown
Owner

전수조사에서 나온 버그 수정.

  • 연락처나 이메일이 빈 학생은 정보 수정이 'Expected string, received null' 로 막히던 문제 (9/18 배포부터 운영에도 있음)
  • 학번 변경 시 인턴십 있으면 HTTP 500 나던 문제
  • 오전 9시 전 상담, 컨택 날짜 기본값이 어제로 들어가던 문제
  • 서버 오류가 빈 목록, 무한 로딩으로 보이던 문제 (10개 화면)
  • 목록 건수 카운트업 제거 (대시보드 지표는 유지)

DB, env 변경 없음.

- 학번 변경 저장 실패: 인턴십 1건 이상인 학생은 HTTP 500 (P2003).
  학번을 먼저 바꾸고 인턴십 삭제, 생성은 옛 학번으로 하던 순서 문제.
  인턴십 0건이면 저장은 되지만 폼에서 지운 인턴십이 남던 문제도 같은 원인
- 오늘 날짜: toISOString 은 UTC 라 오전 9시 전 상담, 컨택 기본값이 어제.
  today() 를 로컬 날짜로 바꾸고 상담 기본값도 today() 사용
- 불러오기 실패 표시: 목록 화면이 SWR error 를 안 받아 서버 오류가
  '조건에 맞는 ~ 없습니다', '불러오는 중…' 멈춤, '학생을 찾을 수 없습니다'
  로 보임. 10개 화면에 홈 대시보드와 같은 '불러오기 실패' 표시 추가.
  캐시된 데이터가 있으면 그대로 유지
- 목록 건수 카운트업 제거: 필터 바꿀 때마다 0부터 다시 올라가 표와 숫자가
  잠깐 안 맞음. 대시보드 지표만 카운트업 유지
- 증상: 연락처나 이메일이 빈 학생은 정보 수정 저장 시 'Expected string,
  received null' (HTTP 422). 학번 변경도 같이 막힘
- 원인: #99 에서 신규 등록에 연락처 필수 추가. 수정 스키마는 partial 이라
  괜찮다고 봤지만 partial 은 '안 보냄' 만 허용하고, 폼은 빈 칸을 null 로 보냄
- 수정 스키마에서 phone, email 을 null 허용으로 다시 풀어 줌. 신규 등록은
  그대로 필수
- 9/18 배포부터 운영에도 있던 문제. 실적 엑셀로 들어온 학생은 연락처가 비어
  있어 대부분 해당

This branch has not been deployed

No deployments
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.

1 participant