Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(mission): add the submission start date time #765

Merged
merged 10 commits into from
Oct 15, 2024

Conversation

woowabrie
Copy link

@woowabrie woowabrie commented Sep 23, 2024

Resolves #768

해결하려는 문제가 무엇인가요?

  • 과제 제출물 제출 시점을 통제하기 위해 시작 일시와 제출 여부를 수동으로 조작해야 하는 불편함이 있습니다.

어떻게 해결했나요?

  • 과제 제출물을 언제 제출할 수 있는지 설정할 수 있도록 제출 시작 일시를 추가하였습니다.
  • 과제 시작 일시, 제출 시작 일시, 종료 일시, 제출 여부를 종합하여 과제 상태를 결정합니다.
    • PREPAREDIN_PROGRESS를 추가하였습니다.
    • SUBMITTABLE을 제거하였습니다.
  • 과제 상태에 따라 UI의 상태를 변경하도록 클라이언트 코드를 수정하였습니다.

어떤 부분에 집중하여 리뷰해야 할까요?

  • 새로운 과제 상태에 따라 UI 상태가 올바르게 표시하는지 확인해 주세요.
  • 과제 상태만으로도 UI 상태를 변경할 수 있으므로 내 과제 목록 조회, 내 과제 조회 API에서 submittable을 제거하였습니다.

참고 자료


RCA 룰

r: 꼭 반영해 주세요. 적극적으로 고려해 주세요. (Request changes)
c: 웬만하면 반영해 주세요. (Comment)
a: 반영해도 좋고 넘어가도 좋습니다. 그냥 사소한 의견입니다. (Approve)

@woowabrie woowabrie self-assigned this Sep 23, 2024
@woowabrie woowabrie changed the title Feature/mission submit period feature(mission): add submit period to mission Sep 23, 2024
Copy link

@woowahan-neo woowahan-neo left a comment

Choose a reason for hiding this comment

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

고생하셨슴다~ 따봉 👍

@@ -42,6 +49,9 @@ class Mission(
@Column(nullable = false)
private var deleted: Boolean = false

val submittable: Boolean
get() = now().isBetween(submitPeriod)

Choose a reason for hiding this comment

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

Suggested change
get() = now().isBetween(submitPeriod)
get() = submitPeriod.contains(now())
// 이름이 애매하다면..? submitPeriod.isWithin(now())

c: MissionSubmitPeriod 내에 확장함수 없애고 위와 같이 작성하는 것이 더 자연스러워 보이는데 어떻게 생각하시나요?

@woowahan-pjs woowahan-pjs force-pushed the feature/mission-submit-period branch from d766594 to bb9c624 Compare October 13, 2024 23:55
Copy link

@woowahan-cron woowahan-cron left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!
frontend에서 특별히 리뷰해야 할 만한 부분은 없었어요.

@woowahan-pjs woowahan-pjs changed the title feature(mission): add submit period to mission feat(mission): add the submission start date time Oct 15, 2024
@woowahan-pjs woowahan-pjs merged commit b9e9d7e into develop Oct 15, 2024
@woowahan-pjs woowahan-pjs deleted the feature/mission-submit-period branch October 15, 2024 13:30
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.

4 participants