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: 출석일을 제외한 날에는 출석 버튼 비활성화 #333

Merged
merged 5 commits into from
Jul 4, 2024

Conversation

abiriadev
Copy link
Member

@abiriadev abiriadev commented Jul 4, 2024

image

💡 다음 이슈를 해결했어요.

  • 스터디 진행 중, 오늘이 스터디 진행일이 아닐 시 출석 버튼을 비활성화합니다.
  • 스터디 종료 시, 출석 버튼을 화면에서 표시하지 않습니다.


💡 이슈를 처리하면서 추가된 코드가 있어요.

  • 오늘이 출석일인지 확인하는 코드 추가
    /**
    * 오늘이 출석일인지 확인한다.
    */
    export const isTodayAttendanceDay = (attendanceDay: Array<AttendanceDay>) =>
    attendanceDay.includes(getDay(new Date()) || 7);
  • 상태에 따라 출석 버튼을 숨기기도, 비활성화할 수 있도록 수정
    {study.status !== 'COMPLETED' &&
    (isTodayAttendanceDay(study.attendanceDay) ? (
    didIAttendToday ? (
    <Button disabled>출석 완료</Button>
    ) : (
    <Button scheme="secondary" onClick={async () => attendStudyMutate()}>
    출석 체크
    </Button>
    )
    ) : (
    <Button disabled>출석 없음</Button>
    ))}



✅ 셀프 체크리스트

  • 브랜치 전략에 맞는 브랜치에 PR을 올리고 있습니다. (master/main이 아닙니다.)
  • 커밋 메세지를 컨벤션에 맞추었습니다.
  • 변경 후 코드는 컴파일러/브라우저 warning/error 가 발생시키지 않습니다.
  • 변경 후 코드는 기존의 테스트를 통과합니다.
  • 테스트 추가가 필요한지 검토해보았고, 필요한 경우 테스트를 추가했습니다.
  • docs 수정이 필요한지 검토해보았고, 필요한 경우 docs를 수정했습니다.

@abiriadev abiriadev self-assigned this Jul 4, 2024
@abiriadev abiriadev added this to the 스프린트 13 milestone Jul 4, 2024
@abiriadev abiriadev merged commit 90cdb1c into dev Jul 4, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant