Skip to content

[Fix] 파일 크기 조정 (#38)#98

Merged
shinae1023 merged 1 commit into
mainfrom
feat/#38-mock-jobposting-with-data
Jun 16, 2026
Merged

[Fix] 파일 크기 조정 (#38)#98
shinae1023 merged 1 commit into
mainfrom
feat/#38-mock-jobposting-with-data

Conversation

@shinae1023

@shinae1023 shinae1023 commented Jun 16, 2026

Copy link
Copy Markdown
Member

✨ 어떤 이유로 PR를 하셨나요?

  • feature 병합
  • 버그 수정(아래에 issue #를 남겨주세요)
  • 코드 개선
  • 코드 수정
  • 배포
  • 기타(아래에 자세한 내용 기입해주세요)

📋 세부 내용 - 왜 해당 PR이 필요한지 작업 내용을 자세하게 설명해주세요

  • 액셀 파일 크기 제한 20mb로 변경

📸 작업 화면 스크린샷

⚠️ PR하기 전에 확인해주세요

  • 로컬테스트를 진행하셨나요?
  • 머지할 브랜치를 확인하셨나요?
  • 관련 label을 선택하셨나요?

🚨 관련 이슈 번호 [ #38]

Summary by CodeRabbit

  • Chores
    • Updated server configuration to set multipart upload limits. Maximum file upload size is now configured to 20MB per request, enabling users to transfer larger files through the application.

@shinae1023 shinae1023 self-assigned this Jun 16, 2026
@coderabbitai

coderabbitai Bot commented Jun 16, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Added spring.servlet.multipart configuration to application.yaml, setting both max-file-size and max-request-size to 20MB.

Changes

Multipart Upload Configuration

Layer / File(s) Summary
Spring multipart upload limits
src/main/resources/application.yaml
Adds max-file-size and max-request-size set to 20MB under the spring.servlet.multipart configuration block.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐇 Hop hop, the rabbit lifts a file,
Twenty megabytes — now that's worth the while!
No more rejections at the gate,
Big uploads welcomed, small or great.
The config whispers: "Come on in!" 🎉

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title '[Fix] 파일 크기 조정 (#38)' translates to '[Fix] File size adjustment (#38)' which directly describes the main change: adjusting file upload size limits to 20MB in the configuration.
Description check ✅ Passed The PR description follows the required template structure with all major sections included. The author selected the bug fix category, provided specific details about the change (Excel file size limit to 20MB), referenced issue #38, and included pre-merge checklist confirmations.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/#38-mock-jobposting-with-data

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 and usage tips.

@shinae1023 shinae1023 merged commit 2ad1c08 into main Jun 16, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant