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

refactor: 미시청 마지막 구간에서 decimal 옵션으로 처리하도록 수정 #221

Merged
merged 1 commit into from
Jun 18, 2024

Conversation

alstjs1207
Copy link
Contributor

PR 의 종류는 어떤 것인가요?

  • 버그 수정
  • 새로운 기능
  • 리팩토링
  • 문서 수정
  • 워크플로우 수정

수정이 필요하게된 이유가 무엇인가요? (Jira 이슈가 있다면 링크를 연결해주세요)

image

콜러스는 클립에 지정된 마감시간보다 더 적게 측정되는 경우가 있다.
이런 경우 강의클립을 다 시청하였더라도 미시청구간이 발생된다.

무엇을 어떻게 변경했나요?

마지막 구간의 경우 반올림 처리하여 미시청구간을 없애도록 한다.

코드 변경을 이해하기 위한 배경지식이 필요하다면 설명 해주세요.

디펜던시 변경이 있나요?

어떻게 테스트 하셨나요?

test code

코드의 실행결과를 볼 수 있는 로그나 이미지가 있다면 첨부해주세요.

@alstjs1207 alstjs1207 requested review from day1-dev-adm and a team June 18, 2024 09:38
@alstjs1207 alstjs1207 self-assigned this Jun 18, 2024
Copy link
Contributor

@jo-seongjun jo-seongjun left a comment

Choose a reason for hiding this comment

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

LGTM!!

@alstjs1207 alstjs1207 merged commit 8ffe2fe into main Jun 18, 2024
2 checks passed
@alstjs1207 alstjs1207 deleted the refactor/unwatched-time-range-decimal-place branch June 18, 2024 23:15
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.

None yet

3 participants