[fix/MAT-632] 일부 학생에서 어드민 알림 발송 에러 관리자 페이지 대응#316
Open
Conversation
- /api/admin/notification/send 400 응답의 message를 ErrorModalTemplate으로 노출 - 발송 성공/실패 무관하게 알림 리스트 invalidate Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/notification페이지에서 어드민 알림 발송 시 백엔드 400 응답을 사용자에게 노출하지 못해 일부 학생만 발송 실패한 케이스를 인지하기 어려웠습니다. 응답의message를ErrorModalTemplate으로 표시하고, 발송 결과(성공/실패) 무관하게 알림 리스트를 갱신하도록 보완했습니다.Linear
Changes
ErrorModalTemplate기반 에러 모달 상태(isErrorModalOpen,errorMessage) 추가handleSendNotificationcatch에서 백엔드 응답message를 추출해 에러 모달로 노출 (없으면 기본 문구 폴백)queryClient.invalidateQueries()를finally블록으로 이동 — 부분 발송 실패 케이스에서도 발송된 알림이 리스트에 반영됨Testing
tsc -b --noEmit—apps/admin/src/routes/_GNBLayout/notification/index.tsx관련 신규 타입 에러 없음 (선재 에러 4건은 본 PR 범위 밖)Risk / Impact
/notification페이지 발송 모달의 catch 경로 및 invalidate 시점message필드가 사용자에게 노출 가능한 텍스트인지 (백엔드 정책 확인)ErrorModalTemplate) 재사용, 호환성 영향 없음Screenshots / Video
백엔드 400 응답 트리거 후 에러 모달 캡처 첨부 예정
🤖 Generated with Claude Code