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.

Sign up for GitHub

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

Jump to bottom

부스트 버그 수정 #263

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

부스트 버그 수정 #263

wants to merge 2 commits into from

Conversation

Zoe0929
Copy link
Collaborator

@Zoe0929 Zoe0929 commented Sep 20, 2024

📌 개요

✍️ 변경사항

  1. 부스트 타이머 시작 전 0.25초 (딤처리 + 뷰 확대) 동안의 지연이 있는데, 이 시기에 탭 제스처를 끝내면 이전 값이 그대로 보내지는 오류가 존재했습니다. 부스트 애니메이션 시작 시 count를 초기화 시키는 로직을 추가하여 수정했습니다.
  2. 부스트 개수가 0이어도 불필요한 서버 통신이 되어 부스트 개수가 0 이상일 때만 서버 통신하도록 수정했습니다.

📷 스크린샷

RPReplay_Final1726840418

🛠️ Technical Concerns(기술적 고민)

@Zoe0929 Zoe0929 added the HotFix label Sep 20, 2024
@Zoe0929 Zoe0929 self-assigned this Sep 20, 2024
@Zoe0929 Zoe0929 linked an issue Sep 20, 2024 that may be closed by this pull request
[HotFix] 부스트 버그 해결 #262
Open
4 tasks
ji-yeon224
ji-yeon224 approved these changes Sep 20, 2024
View reviewed changes
Copy link
Collaborator

@ji-yeon224 ji-yeon224 left a comment

Choose a reason for hiding this comment

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

고생해써요~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ji-yeon224 ji-yeon224 ji-yeon224 approved these changes

@usa4060 usa4060 Awaiting requested review from usa4060

Assignees

@Zoe0929 Zoe0929

Labels
HotFix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[HotFix] 부스트 버그 해결
2 participants
@Zoe0929 @ji-yeon224