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] #231 - 기존 결석 케이스 지각으로 수정 #359

Merged
merged 2 commits into from
Mar 31, 2024

Conversation

0inn
Copy link
Contributor

@0inn 0inn commented Mar 30, 2024

🌴 PR 요약

🌱 작업한 브랜치

🌱 PR Point

기존 결석 케이스를 지각 케이스로 수정했습니다.

AS-IS
[출석, 결석] -> 결석

TO-BE
[출석, 결석] -> 지각

📌 참고 사항

이번 기수에서 기존 결석 케이스를 지각으로 가져간다고 해서 대응했습니다.

현재 서버에서 내려오는 [1차 출석, 2차 출석] 상태에 따라 [최종 출석]을 클라이언트 측에서 계산해주고 있는데요.
이에 따라서 나올 수 있는 경우의 수는 아래와 같습니다.

1차 출석 2차 출석 최종 출석
출석 결석 지각
결석 결석 결석
결석 출석 지각
출석 출석 출석

이에 맞게 ShowAttendanceUseCase 쪽 로직 수정 및 주석 추가해뒀습니다.
아무래도 빠르게 대응해야 하는 건이라 실례일수 있는데 제가 진행하는 점 죄송합니다. (꾸벅)

📸 스크린샷

기능 스크린샷
지각 케이스

📮 관련 이슈

@0inn 0inn added Feat 새로운 기능 구현 영인☁️ labels Mar 30, 2024
@0inn 0inn requested review from elesahich, ffalswo2 and lsj8706 March 30, 2024 08:06
@0inn 0inn self-assigned this Mar 30, 2024
Copy link

height bot commented Mar 30, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Copy link
Member

@lsj8706 lsj8706 left a comment

Choose a reason for hiding this comment

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

빠른 대응 감사합니다~~!!

Copy link
Contributor

@elesahich elesahich left a comment

Choose a reason for hiding this comment

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

작업 감사합니다 영인~

@ffalswo2
Copy link
Contributor

감사합니다!!

Copy link
Member

@devxsby devxsby left a comment

Choose a reason for hiding this comment

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

고생하셨습니당..
혹시 지각 , 지각(조퇴) 는 없겠죠..? ㅎㅎ..;;

@0inn
Copy link
Contributor Author

0inn commented Mar 31, 2024

@devxsby
마지막 지각 상태 네이밍이 바뀔 수 있다는 의민가여 ?!
(일단 머지할게요 ^~^)

@0inn 0inn merged commit 5c21b8e into sopt-makers:develop Mar 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 새로운 기능 구현 size/M 영인☁️
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 기존 결석 케이스 지각으로 수정
5 participants