Skip to content

Feat/mission-final - #23

Merged
chaeyylee merged 2 commits into
mainfrom
feat/mission-final
Jul 10, 2026
Merged

Feat/mission-final#23
chaeyylee merged 2 commits into
mainfrom
feat/mission-final

Conversation

@chaeyylee

@chaeyylee chaeyylee commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • 버그 수정

    • 미션 목표 날짜가 데드라인과 정확히 일치할 때 날짜가 잘못 계산되던 문제를 수정했습니다.
    • 데드라인 시점에는 이전 날짜로, 데드라인 이후에는 현재 날짜로 정확히 처리됩니다.
  • 개선 사항

    • 이미지 업로드 결과 URL이 설정된 저장 리전에 맞게 생성되도록 검증을 보완했습니다.

@chaeyylee chaeyylee self-assigned this Jul 10, 2026
@coderabbitai

coderabbitai Bot commented Jul 10, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 68a15daa-e16b-437e-be60-3253e31b7156

📥 Commits

Reviewing files that changed from the base of the PR and between b9dcff4 and 72eb647.

📒 Files selected for processing (3)
  • src/main/java/com/example/hackathon/domain/mission/service/MissionTargetDateResolver.java
  • src/test/java/com/example/hackathon/domain/image/service/S3StorageServiceTest.java
  • src/test/java/com/example/hackathon/domain/mission/service/MissionTargetDateResolverTest.java

📝 Walkthrough

Walkthrough

미션 날짜 resolver가 정확한 데드라인을 이전 날짜로 처리하도록 변경되었고, 경계 조건 테스트가 추가되었습니다. S3 업로드 URL 테스트는 us-west-2 리전에 맞게 갱신되었습니다.

Changes

미션 날짜 경계 처리

Layer / File(s) Summary
데드라인 날짜 해석 변경
src/main/java/.../MissionTargetDateResolver.java, src/test/java/.../MissionTargetDateResolverTest.java
resolve가 정확히 데드라인인 경우 이전 날짜를 반환하고, 데드라인 직후에는 현재 날짜를 반환하도록 조건과 테스트를 변경했습니다.

S3 리전 URL 테스트

Layer / File(s) Summary
S3 리전 기대값 갱신
src/test/java/.../S3StorageServiceTest.java
테스트 리전을 us-west-2로 변경하고 업로드 URL 접두사 기대값을 갱신했습니다.

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

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive 제목이 너무 일반적이라 변경 내용을 구체적으로 알 수 없습니다. 주요 변경 사항을 드러내는 구체적인 제목으로 수정하세요. 예: 미션 목표일 계산 로직 수정.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/mission-final

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

@chaeyylee
chaeyylee merged commit bd35dde into main Jul 10, 2026
2 checks passed
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