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

미션 상태 - 인원 미달, 종료 대기 대응 #73

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from

Conversation

eshc123
Copy link
Member

@eshc123 eshc123 commented Nov 8, 2024

주요 내용

  • 미션 상태를 나타내는 MissionStatus enum class를 추가하였습니다.
  • /api/mission-members/me 호출 시 CREATED, CANCELED, IN_PROGRESS, PENDING_COMPLETION 을 필터 값으로 두고 CREATED, IN_PROGRESS 은 보드 화면, CANCELED은 보드 화면 후 인원 미달 다이얼로그, PENDING_COMPLETION 은 미션 종료 화면으로 이동합니다.
  • 미션 종료 화면(결승선 화면)에서 확인 버튼 클릭 시 미션 완료 API를 호출하여 미션 상태를 완료로 변경 요청 합니다.
    KakaoTalk_Video_2024-11-09-00-11-33-ezgif com-resize

@eshc123 eshc123 requested a review from bywindow November 8, 2024 15:16
@eshc123 eshc123 self-assigned this Nov 8, 2024
Comment on lines +3 to +10
enum class MissionStatusResponse {
CREATED,
CANCELED,
IN_PROGRESS,
DELETED,
PENDING_COMPLETION,
COMPLETED
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

enum을 리스폰스로 받거나 리퀘스트로 json 파싱해서 넘겨줄 때 SerializedName 쓰거나 JsonAdapter 사용하는 방식도 있는데, 우리가 kotlin-serialize 사용하고 있어서 그 방법으로는 못하는거야?

Copy link
Member Author

Choose a reason for hiding this comment

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

못한다기보다는 안 써도 되는 형태라고 봐야하지 않을까 싶어! https://github.com/Kotlin/kotlinx.serialization/blob/master/docs/builtin-classes.md#enum-classes
보면 enum class name이랑 동일한 경우 그대로 사용해도 되고 다른 경우에는 @SerialName 를 통해 지정을 해도 되는 형태인 것 같아.
필드값으로 들어가는 enum class 에 대해서는 @serializable annotation을 안 써도 되는지에 대해서 확실하게 몰랐는데 공식 가이드에서 안 써도 된다고 적혀있네!

…atus

# Conflicts:
#	core/data/mission/src/main/java/com/goalpanzi/mission_mate/core/board/repository/MissionRepositoryImpl.kt
#	core/domain/mission/src/main/java/com/goalpanzi/mission_mate/core/domain/mission/repository/MissionRepository.kt
#	core/network/src/main/java/com/goalpanzi/mission_mate/core/network/service/MissionService.kt
@eshc123 eshc123 deployed to Android CI/CD November 12, 2024 11:39 — with GitHub Actions Active
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.

2 participants