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

[DA] 마감임박 타이머 만료시 이슈 대응 #191

Merged
merged 3 commits into from
Aug 26, 2022

Conversation

szzang-kr
Copy link
Contributor

개요

변경사항

작업 내용

  • 마감임박 타이머 만료시 binding 문제로 인한 무수히 많은 reload를 대응하였습니다. 12adf3f
    • 마감임박 기프티콘 데이터가 새로 셋팅 될 때 마다 disposeBag을 초기화 하여 binding 된 내용을 dispose하였습니다.
  • 마감임박 타이머 만료 시 카테고리별 기프티콘 리스트는 새로 받아오지 않아 리스트에 그대로 남아 있는 현상을 대응하였습니다. eef1b22
  • 마감임박 타이머 만료 시 API를 새로 요청하지만, 데이터가 그대로 남아 있는 현상을 대응하였습니다. 1bbacd6
    • 마감 시간이 현재시간보다 지난 경우에는 마감임박 기프티콘 리스트에 추가 하지 않도록 방어코드를 추가했습니다.
    • 그러나 API 재요청시 현재시간보다 미세하게 빨라 리스트에 그대로 노출되는 현상이 발생했습니다.
    • 위 이슈를 해결하기 위해 API 요청 전에 delay를 추가하여 조금 늦게 API를 요청하도록 하여 해결하였습니다.

미리보기

작업 전

2022-08-26.1.10.30.mov

작업 후

2022-08-26.1.03.41.mov

Copy link
Collaborator

@kimkyunghun3 kimkyunghun3 left a comment

Choose a reason for hiding this comment

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

고생했습니다!

Comment on lines +37 to +38
.skip(1)
.throttle(.milliseconds(500), scheduler: MainScheduler.instance)
Copy link
Collaborator

Choose a reason for hiding this comment

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

요거 궁금한건데 skip이 지정한 횟수만큼 skip를 하는거라고 본거같은데 이게 왜 필요할까요?!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

BehaviorRelay는 subscribe시 가지고 있는 값을 방출하는 Hot Observable입니다.

따라서 해당 이벤트를 무시하기 위해 .skip(1)을 추가하였습니다.

자세한 내용은 아래 자료를 참고하시면 좋습니다. :)
https://okanghoon.medium.com/rxswift-2-observable-subject-relay-8fcd9b01913d

Copy link
Collaborator

@AhnSangHee AhnSangHee left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 대장님👍

@szzang-kr szzang-kr merged commit 420f110 into develop Aug 26, 2022
@szzang-kr szzang-kr deleted the feature/sz/deadlinecell-refactor branch August 26, 2022 10:00
@szzang-kr szzang-kr mentioned this pull request Aug 26, 2022
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.

3 participants