Skip to content

fix: 사이트용 신청 폼 PUT 엔드포인트 삭제 - #435

Merged
kimjiy9607 merged 3 commits into
betafrom
fix/remove-site-application-form-put
Nov 8, 2026
Merged

kimjiy9607 merged 3 commits into
betafrom
fix/remove-site-application-form-put

Conversation

@kimjiy9607

Copy link
Copy Markdown
Contributor

네비게이터가 PUT /api/studies/{studyId}/application-form 으로 신청 폼을 덮어쓸 수 있는 문제를 수정한다.

specs/authz-guards/spec.md B절 후속(2026-10-04 기획 확정)에 따라 사용자 사이트 PUT 을 삭제한다. 신청 폼 저장은 백오피스
PUT /api/admin/studies/{studyId}/application-form (@RequireAdmin, 캡틴만) 단일 경로로 통일.

요약

변경 사항

체크리스트

Self Review

  • 내 코드를 직접 리뷰했다 (/self-review 실행 또는 수동)
  • 불필요한 console.log / println / 디버그 코드를 제거했다
  • 변수·함수 이름이 의도를 명확히 드러낸다
  • 에러 핸들링이 빠진 경로가 없다

테스트

  • Stage 에서 테스트했다 (해당 시 스크린샷 첨부)
  • 단위/통합 테스트를 추가 또는 업데이트했다
  • 기존 테스트가 깨지지 않는다 (./gradlew test / pnpm test)

보안 (PUBLIC 레포)

  • 시크릿·토큰·API 키·PII 가 코드에 없다
  • .env 파일이 커밋에 포함되지 않았다

기타

  • 스키마 변경 시 V{n}__*.sql 마이그레이션을 같이 넣었다
  • 관련 문서(ERD, 스펙)를 업데이트했다
  • 01-planning/stories/** 또는 frontend/apps/playground/**/spec.ts 를 건드렸다면 pnpm --filter playground export-specs --check 통과 확인

팀 공유

  • 팀이 알아야 할 결정이 생겼다 → docs/share/ 에 한 장 (없으면 지운다)
  • prod 사고 수정이다 → docs/incidents/ 에 한 장 (없으면 지운다)

네비게이터가 PUT /api/studies/{studyId}/application-form 으로 신청 폼을
덮어쓸 수 있는 문제를 수정한다.

specs/authz-guards/spec.md B절 후속(2026-10-04 기획 확정)에 따라
사용자 사이트 PUT 을 삭제한다. 신청 폼 저장은 백오피스
PUT /api/admin/studies/{studyId}/application-form (@RequireAdmin, 캡틴만) 단일 경로로 통일.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@kimjiy9607
kimjiy9607 requested a review from titaniper as a code owner November 7, 2026 23:48
Jiyoon Kim and others added 2 commits November 7, 2026 15:56
- 네비게이터 사이트 저장 성공 케이스 → 삭제된 엔드포인트 4xx 확인으로 교체
- 외부인 거부·잠긴 폼·예약 질문 ID 테스트 → 백오피스 경로(캡틴)로 이동

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
B절 표의 신청 폼 저장 행 "이번" 열을 "삭제 완료(2026-11-07)"로,
HOW절 스터디 단위 적용 대상에서 삭제된 사이트 경로를 명시 제거.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@kimjiy9607
kimjiy9607 merged commit 67fb650 into beta Nov 8, 2026
3 checks passed
@kimjiy9607
kimjiy9607 deleted the fix/remove-site-application-form-put branch November 8, 2026 00:03
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