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

Feat/#245 홈 / 알람 쓰로틀, 무한스크롤 적용 #312

Merged
merged 6 commits into from
May 26, 2024
Merged

Feat/#245 홈 / 알람 쓰로틀, 무한스크롤 적용 #312

merged 6 commits into from
May 26, 2024

Conversation

gnksbm
Copy link
Contributor

@gnksbm gnksbm commented May 19, 2024

MUKER-WON and others added 5 commits April 26, 2024 14:19
@MUKER-WON
[Feat] 알람 -> 정류장: 쓰로틀 작업 (#261)
91c5b91 
* [Feat] #260 BusStop Refresh 제한 설정

* [Fix] refreshLoading 옵셔널 변경

* [Feat] viewWillAppear Flow에 대한 분기처리 추가
@MUKER-WON
[Feat] BusStopFeature Refresh Throttle 15초 처리
44b2df4
@MUKER-WON
Revert "[Feat] BusStopFeature Refresh Throttle 15초 처리"
e0c92cb 
This reverts commit 44b2df4.
@MUKER-WON
Feat/#267 BusStopFeature Refresh Throttle 처리 (#268)
804c66a 
* [Feat] BusStopFeature Refresh Throttle 15초 처리

* [Add]  viewModel.flw 분기처리 수정

* [Chore] refresh 쓰로틀 시간 3초로 변경

* [Add] willEnterForegroundNotification시점 추가

* [Style] 쓰로틀 상태 Enum 추가

* [Chore] Camel Case 수정
@gnksbm @MUKER-WON
Feat/#255 즐겨찾기 Fetch 페이지네이션 처리 (#265)
05d37ed 
* [Feat] #251 즐겨찾기 정보 Fetch 요청 쓰로틀 작업

* [Remove] VC 레거시 코드 제거

* [Fix] RefreshController 애니메이션 시작 / 종료 버그 수정

* [Fix] 홈 화면 타이머 버그 수정

* [Feat] 페이지네이션 fetch 함수 구현

* [Add] 페이지네이션 fetch 범위 수정

* [Chore] 디버깅 로그 수정

* [Feat] 즐겨찾기 Fetch 페이지네이션 적용

* [Fix] BCTimer 로직 수정

* [Fix] 버스도착정보 모델 date 추가 및 시간정보 update 로직 수정

* [Chore] pagenation 후 중복 제거 코드 삭제

* [Fix] 업데이트 버튼 시간 정보 버그 수정

* [Chore] Lint에러 수정

* [Fix] 즐겨찾기 Fetch 방식 수정, 홈 화면 즐겨찾기된 정류장 표시 버그 수정

* [Fix] #265 중복된 즐겨찾기 버그 수정

* [Fix] #265 중복된 즐겨찾기 버그 수정

* [Chore] 레거시 함수 제거 및 주석 추가

* [Fix] #265 정류장 Fetch 버그 수정

#265 (comment)

* [Remove] 레거시 코드 제거

* [Fix] FavoritesRepository 마이그레이션 로직 수정

* [Fix] #265 홈화면 즐겨찾기 변화에 따른 FakeFetching 로직 수정

* [Chore] #265 즐겨찾기 스크롤이벤트 변수명 수정

* [Fix] #265 버스도착정보 시간정보 업데이트 로직 수정

* [Chore] 디버그 print문 삭제

---------

Co-authored-by: MUKER-WON <[email protected]>
@gnksbm gnksbm added the ✨ 기능추가 새로운 기능 추가 label May 19, 2024
@gnksbm gnksbm requested a review from MUKER-WON May 19, 2024 01:48
@gnksbm gnksbm linked an issue May 19, 2024 that may be closed by this pull request
[Feat] 앱 쓰로틀 처리 작업 #245
Closed
4 tasks
@MUKER-WON
Copy link
Contributor

@gnksbm
설정된 쓰로틀대로 잘 동작하는걸로 보입니다
문제점 발견하신게 있으신가요?

@gnksbm
Copy link
Contributor Author

gnksbm commented May 20, 2024

@MUKER-WON
쓰로틀 구현을 동일한 형태로 수정하는 건 어떠실까요?
이번 PR Merge 전이나 예정된 리팩토링때 하면 좋을 것 같습니다

@MUKER-WON
Copy link
Contributor

@gnksbm
쓰로틀 형태를 맞추는거 좋습니당
리팩토링할 때 쓰로틀을 처리하는 부분의 로직이 바뀔거 같다면 그 때 수정하는게 좋을거 같구요
변하지 않을거면 merge전에 하면 좋을거 같은데, 저는 바뀔 확률이 다분하다 생각해서 리팩토링 때 맞춰서 바꾸는것도 괜찮은거 같습니다
건섭님은 어떠세요?

@yuhaeun-la
Copy link
Contributor

제목 간단하게 적어주시면 좋을 것 같습니다!

@gnksbm
Copy link
Contributor Author

gnksbm commented May 23, 2024

@MUKER-WON
변할 확률이 높지만 동일한 형태라면 리팩토링이 좀 더 수월할 것 같습니다
다만 리팩토링 계획이 세워져야 방향을 알 수 있을 것 같아 추후 작업하는 방향이 좋아보여요!

@gnksbm gnksbm changed the title Feat/#245 Feat/#245 홈 / 알람 쓰로틀, 무한스크롤 적용 May 23, 2024
@MUKER-WON MUKER-WON merged commit cc4221a into dev May 26, 2024
1 check passed
@MUKER-WON MUKER-WON deleted the feat/#245 branch May 26, 2024 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MUKER-WON MUKER-WON Awaiting requested review from MUKER-WON

Assignees

@MUKER-WON MUKER-WON

@gnksbm gnksbm

Labels
✨ 기능추가 새로운 기능 추가
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feat] 앱 쓰로틀 처리 작업
3 participants
@gnksbm @MUKER-WON @yuhaeun-la