Skip to content

Fix(#151): 자료 업로드 주차 자동 지정 및 폼 디자인 수정 - #152

Merged
BigSangHyeok merged 1 commit into
devfrom
fix/151-upload-auto-week
Aug 13, 2026
Merged

Fix(#151): 자료 업로드 주차 자동 지정 및 폼 디자인 수정#152
BigSangHyeok merged 1 commit into
devfrom
fix/151-upload-auto-week

Conversation

@BigSangHyeok

@BigSangHyeok BigSangHyeok commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

📌 관련 이슈

🏷️ PR 타입

  • ✨ 기능 추가 (Feature)
  • 🐛 버그 수정 (Bug Fix)
  • ♻️ 리팩토링 (Refactoring)
  • 🎨 스타일 변경 (Style)
  • ✅ 테스트 추가 (Test)
  • ⚙️ 프로젝트 초기 설정 (Chore)
  • 📝 문서 수정 (Documentation)
  • 🚨 PR 복구 (Revert)

📝 작업 내용

  • 자료 업로드 폼의 주차 선택 UI 제거
  • 업로드 요청 및 multipart 데이터에서 week 제거
  • 업로드 시점 기준 자동 주차 지정 정책 반영
  • 관련 타입과 테스트 수정

📸 스크린샷

✅ 체크리스트

  • 코드 리뷰를 받을 준비가 완료되었습니다.
  • 코드 스타일 가이드를 준수했습니다.
  • 셀프 리뷰를 완료했습니다.
  • 테스트를 작성하고 모두 통과했습니다.
  • 문서를 업데이트했습니다. (필요한 경우)
  • CI의 format, lint, build 검사를 모두 통과했습니다.
  • 최신 dev 브랜치를 현재 작업 브랜치에 반영했습니다.

🔀 Merge 규칙

  • 기능 브랜치에서 dev로 병합할 때는 Squash and merge를 사용합니다.
  • 배포 시 dev에서 main으로 병합할 때는 Create a merge commit을 사용합니다.
  • Rebase and merge는 사용하지 않습니다.
  • CI 통과와 최신 dev 반영을 확인한 후 Merge합니다.

📎 기타 참고사항

  • 업로드 자료의 주차는 백엔드에서 업로드 시점을 기준으로 자동 지정합니다.

Summary by CodeRabbit

  • 변경 사항
    • 자료 업로드 시 주차를 선택하는 기능을 제거했습니다.
    • 자료 등록 시 제목, 본문, 설명 및 파일만 입력·전송됩니다.
    • 업로드 화면에서 주차 관련 검증과 입력 항목이 사라졌습니다.
    • 업로드 완료 후 기존 성공 처리와 자료 목록 갱신 동작은 유지됩니다.

@BigSangHyeok BigSangHyeok self-assigned this Aug 13, 2026
Copilot AI lite review requested due to automatic review settings August 13, 2026 11:43
@BigSangHyeok BigSangHyeok added the bug 버그 수정 label Aug 13, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 02d53cd7-59ef-4fb2-88ce-3e01d60751d9

📥 Commits

Reviewing files that changed from the base of the PR and between 773736f and c3a9df1.

📒 Files selected for processing (7)
  • src/pages/study/StudyPage.tsx
  • src/pages/study/upload/MaterialUploadForm.tsx
  • src/pages/study/upload/MaterialUploadPage.test.tsx
  • src/pages/study/upload/MaterialUploadPage.tsx
  • src/pages/study/upload/hooks/useSubmitMaterial.ts
  • src/shared/api/upload.ts
  • src/shared/types/stology.ts
💤 Files with no reviewable changes (4)
  • src/pages/study/upload/hooks/useSubmitMaterial.ts
  • src/shared/api/upload.ts
  • src/shared/types/stology.ts
  • src/pages/study/StudyPage.tsx

📝 Walkthrough

Walkthrough

자료 업로드 폼에서 주차 선택과 검증을 제거했습니다. 업로드 요청과 관련 타입에서 week를 제거했습니다. 페이지 연결과 테스트도 새 동작에 맞게 갱신했습니다.

Changes

자료 업로드 주차 제거

Layer / File(s) Summary
업로드 계약과 요청 payload
src/shared/types/stology.ts, src/shared/api/upload.ts, src/pages/study/upload/hooks/useSubmitMaterial.ts
MaterialDraftUploadMaterialReq에서 week를 제거했습니다. multipart 요청과 제출 payload에서도 week를 전송하지 않습니다.
주차 없는 업로드 폼
src/pages/study/upload/MaterialUploadForm.tsx
주차 상태, 선택 UI, 동적 검증, 초기값, 제출 데이터 및 초기화 처리를 제거했습니다.
페이지 연결과 테스트 갱신
src/pages/study/StudyPage.tsx, src/pages/study/upload/MaterialUploadPage.tsx, src/pages/study/upload/MaterialUploadPage.test.tsx
페이지 props와 폼 호출에서 currentWeek를 제거했습니다. 테스트는 주차 선택 UI 부재와 week 미전송을 검증합니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Mergeability Score: 🔵 Low · up to c3a9d

The upload flow now relies on the server to assign the material week instead of accepting it from the browser. The PR is mergeable with explicit owner awareness that the deployed API must accept requests without week and must ignore or reject client-supplied week values so uploads do not fail or bypass the intended assignment policy.

Possibly related PRs

  • Stology/FE#52: 현재 변경이 수정하는 자료 업로드 폼과 타입을 도입한 PR입니다.
  • Stology/FE#107: 자료 업로드 페이지와 요청 관련 파일을 함께 수정한 PR입니다.
  • Stology/FE#150: 자료 업로드의 week 필드를 직접 수정한 PR입니다.

Suggested labels: enhancement, style

Suggested reviewers: jea7

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning 주차 제거와 미전송 검증은 반영했지만 2열 레이아웃과 오른쪽 상단 배치 변경은 확인되지 않습니다. 파일·텍스트 입력 폼을 2열로 정렬하고 제목·설명·등록 버튼을 오른쪽 상단에 배치하세요.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed 제목은 자료 업로드 주차 자동 지정과 폼 수정이라는 주요 변경 목적을 명확하게 설명합니다.
Description check ✅ Passed 필수 섹션과 주요 변경 내용, 체크리스트를 작성했으며 스크린샷만 비어 있습니다.
Out of Scope Changes check ✅ Passed 변경 사항은 주차 선택 제거, 업로드 요청 수정, 타입 변경, 테스트 수정으로 연결 이슈 범위에 포함됩니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@BigSangHyeok

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@mint0326 mint0326 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다

@BigSangHyeok
BigSangHyeok merged commit a65e3e2 into dev Aug 13, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 버그 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fix] 자료 업로드 주차 자동 지정 및 폼 디자인 수정

3 participants