LC-3128 무료 자료집 인플루언서 전용 신청 수정#2403
Merged
Hidden character warning
The head ref may contain hidden characters: "LC-3128-\ubb34\ub8cc-\uc790\ub8cc\uc9d1-\uc778\ud50c\ub8e8\uc5b8\uc11c-\uc804\uc6a9-\uc2e0\uccad-\uc218\uc815"
Merged
Conversation
BE의 isAccessible(접속 가능) 분리에 맞춰 어드민 토글·웹 신청 차단을 연결. [어드민] - 마그넷 목록/상세 스키마·타입에 isAccessible 추가(백필 전 null→false 변환) - usePatchMagnetAccessibilityMutation 추가, "접속 가능" 체크박스 컬럼 신설 - 상세/신청 링크 복사 게이트를 isVisible → isAccessible 로 변경 (목록 비노출이어도 링크로 접속 가능한 인플루언서 케이스 지원) - "노출여부" 라벨을 "목록 노출"로 정리 [웹] - 신청 제출 403(MAGNET_NOT_ACCESSIBLE)을 "신청 마감" 안내 모달로 분기 (예상된 비즈니스 차단 → Sentry 미보고) BE: Let-s-intern/lets-career-server#817 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Code Review
이번 풀 request는 마그넷의 목록 노출 여부와 독립적으로 링크 접속 및 신청 가능 여부를 제어할 수 있는 isAccessible 필드와 관련 API, 스키마, 타입, 목 데이터 및 어드민 UI(체크박스 토글 기능)를 추가하고, 웹 서비스에서 접속 불가 시 403 에러 처리를 추가하는 변경사항을 담고 있습니다. 리뷰어는 403 에러 발생 시 상세 페이지 대신 전체 목록 페이지(/library)로 안전하게 이동하도록 라우팅을 개선할 것과, magnetSchema.ts 내 중복되는 isAccessible 변환 로직을 공통 헬퍼로 분리하여 재사용성을 높일 것을 권장했습니다.
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.
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
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.
개요
BE의 노출여부 2축 분리(목록 노출
isVisible/ 접속 가능isAccessible)에 맞춰 어드민 토글과 웹 신청 차단 안내를 연결합니다.목록 노출 OFF + 접속 가능 ON으로 설정 → 무료 자료집 목록엔 안 뜨지만 링크로 접속·신청 가능 (인플루언서 전용 페이지).변경
어드민
api/magnet/magnetSchema.ts: 목록·상세 스키마에isAccessible추가 (BE 백필 전 null/undefined → false 변환으로 안전)api/magnet/magnet.ts:PatchMagnetReqBody에isAccessible,usePatchMagnetAccessibilityMutation추가domain/admin/magnet/types.ts:MagnetListItem/MagnetPostDetail에isAccessiblepages/magnet/MagnetTable.tsx: "접속 가능" 체크박스 컬럼 신설, "노출여부"→"목록 노출" 라벨 정리, 상세/신청 링크 복사 게이트를isVisible→isAccessible로 변경 (목록 비노출이어도 링크 복사 가능)pages/magnet/MagnetListPage.tsx: 접속 가능 토글 핸들러 연결*/magnet/mock.ts×2: 타입 정합용isAccessible추가웹
domain/library/apply/MagnetApplyContent.tsx: 신청 제출 403(MAGNET_NOT_ACCESSIBLE)을 "신청이 마감되었어요" 모달로 분기 (예상된 차단 → Sentry 미보고)동작
검증
참고
🤖 Generated with Claude Code