Skip to content

Conversation

@zzaekkii
Copy link
Member

close #169

☑️ 완료 태스크

  • 수동 마감 기능 추가
  • 테스트 코드 작성

🔎 PR 내용

PR 리뷰 요청글 등록 시 설정한 마감 시각이 되기 전에 미리 수동 마감할 수 있도록 하는 기능 추가했습니다.

나중에 자동 마감 배치가 추가될 텐데
이때 만약 서로 근소한 차이를 두고 자동 마감 배치와 수동 마감 작업이 들어와 경합이 발생할 수 있으므로,
이미 (자동 마감 배치로) 마감 상태인 요청글에 수동 마감 요청이 후속으로 들어오면
수동 마감 요청이 들어온 시각이 마감 시각보다 이르면 수동 마감 시각을 우선하여 갱신하는 쪽으로 했습니다.

요약: 수동 · 자동 경합 시, 수동 마감 시각을 우선한다.

이런 상황을 고려해서, 이미 마감된 글에 수동 마감 API 호출이 되더라도 예외 처리되진 않습니다.
자동 마감 배치에서 예외가 발생하면 곤란하기 때문에, 예외 대신 아무 작업도 하지 않는 방향으로 했습니다.


@zzaekkii zzaekkii requested a review from seungryul99 November 13, 2025 07:45
@zzaekkii zzaekkii self-assigned this Nov 13, 2025
@zzaekkii zzaekkii added ✅ Test test 관련(storybook, jest...) :trollface: 재영 Further information is requested labels Nov 13, 2025
@zzaekkii zzaekkii linked an issue Nov 13, 2025 that may be closed by this pull request
2 tasks
@zzaekkii zzaekkii added the ✨ Feature 기능 개발 label Nov 13, 2025
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
13.3% Coverage on New Code (required ≥ 80%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@zzaekkii zzaekkii merged commit 889d91f into develop Dec 5, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Feature 기능 개발 size/L ✅ Test test 관련(storybook, jest...) :trollface: 재영 Further information is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feat: 요청글 수동 마감 기능 추가

2 participants