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

Fix checking first attendance api #124

Merged
merged 1 commit into from
Aug 10, 2023
Merged

Conversation

mkSpace
Copy link
Collaborator

@mkSpace mkSpace commented Aug 6, 2023

첫 참석자 조회할 때 status 여부를 고려하지 않게 번경했습니다

Copy link
Contributor

@SeongYunKim SeongYunKim left a comment

Choose a reason for hiding this comment

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

아 첫 출석자가 지각이나 결석일 수도 있겠군요. LGTM 입니다! 👍

Copy link
Member

@KimDoubleB KimDoubleB left a comment

Choose a reason for hiding this comment

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

LGTM 👍🏻

@mkSpace mkSpace merged commit 046003f into develop Aug 10, 2023
@mkSpace mkSpace deleted the feature/fix_first_attendance branch August 10, 2023 11:48
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