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

[최종] 개발 사항 배포 #347

Merged
merged 169 commits into from
Dec 2, 2021
Merged

[최종] 개발 사항 배포 #347

merged 169 commits into from
Dec 2, 2021

Conversation

tmfrlrkvlek
Copy link
Collaborator

그동안 BBus를 사랑해주신 여러분 감사합니다 🥲

작업 내용

  • 리팩토링
  • 테스트
  • bug fix

시연 방법

기타 (고민과 해결, 리뷰 포인트 등)

tmfrlrkvlek and others added 30 commits November 29, 2021 16:51
@tmfrlrkvlek
refactor: queue 제거, retry 예제 추가, 정거장 화면 로더 및 즐겨찾기 이슈 해결
d54ec91
fix: station 즐겨찾기 시점문제 해결
b45ae97
add: UseCase 작업 전 틀 나누기
a9e75db
@minsangKang
refactor: #298 BusRouteUseCase AnyPublisher 타입반환 리펙토링
25a20a2
@tmfrlrkvlek
refactor: #300 반환타입 Publisher로, 프로토콜화, loader 로직 수정
c292d24
@minsangKang
refactor: #299 MovingStatusUseCase AnyPublisher 타입반환 리펙토링
6d42a4d
@minsangKang
refactor: #298 protocol 누락 추가
9ac28bc
refactor: #302 UseCase Publisher 반환하도록 수정, Protocol 채택
67e0823
@Modyhoon
Merge pull request #304 from boostcampwm-2021/issue/#300
73efd5e 
[리팩토링] AlarmSetting 화면 UseCase 리팩토링
@Modyhoon
Merge pull request #305 from boostcampwm-2021/issue/#298
af7ca04 
[리펙토링] BusRouteUsecase 화면 UseCase 리펙토링
@Modyhoon
Merge pull request #307 from boostcampwm-2021/issue/#299
bc27901 
[리펙토링] MovingStatusUsecse 화면 UseCase 리펙토링
@Modyhoon
Merge branch 'develop' into issue/#302
35bbfa2
@Modyhoon
Merge pull request #308 from boostcampwm-2021/issue/#302
28c90d2 
[리팩토링] 홈화면 UseCase리팩토링
@sujeong000
rename: #306 StationUseCase -> StatoinAPIUseCase
f987c77
@minsangKang
refactor: #297 MovingStatus viewModel -> useCase 로직 이동
90e9cd1
@minsangKang
refactor: #297 NetworkError 처리로직 UseCase -> ViewModel 로 이동
010edc1
@nonalias
refactor: #309 custom refresh button 생성
f182f59
refactor: #309 홈 화면 RefreshableView 채택 완료
8a7a844 
1. 참고로, HomeView의 addSubviews 내에 refresh Button을 한 번 더 추가해준 이유는 순서조정때문이다. RefreshButtonView 내에 layout을 설정하기 위해서는 반드시 addSubView를 해주어야하는데, 선행되므로 뷰 밑에 쌓이게 된다. 따라서 최상위로 올려주기 위해 다시 추가해 주었다. 어차피 뷰는 레퍼런스타입이기에 중복하여 추가하면 순서만 바뀐다.
@minsangKang
fix: #297 rebase 오류 해결 (alarmSettingViewModel 메소드 추가)
207ace2
refactor: 왜 있는지 모르겠는 statusBarView 제거.
77686ec 
제거해도 뷰엔 별 차이 없음. 아마 상단 시계 있는 쪽 색상 지정하려고 한것같은데, 없애도 별차이없음. 다크모드때도 마찬가지.
@minsangKang
refactor: #297 #299 NetworkError 처리 로직 UseCase -> ViewModel 로 이동
ff99b53
@tmfrlrkvlek
refactor: #303 Service, Persistence 이름 변경, TokenManager, RequestFacto…
836379c 
…ry 분리
@tmfrlrkvlek
refactor: #303 Service, Persistence 프로토콜화, Service, Persistence struc…
4f2f39d 
…t로 변경
@tmfrlrkvlek
fix: #303 rebase develop UseCase 명칭 관련 conflict 해결
bd694b7
@minsangKang
refactor: #312 BaseViewControllerType 프로토콜 추가
ac36ca1
@sujeong000
refactor: #306 리턴타입 Publisher로 변경
0213fc0
@sujeong000
refactor: #306 Calculate UseCase 추가 및 분리
75a4488
@sujeong000
restyle: #306 noInfoAlert 메세지 수정
9b53188
@sujeong000
refactor: #301 API UseCase 프로토콜화
99d5876
@sujeong000
rename: #306 load -> bind
1c05cd4
sujeong000 and others added 22 commits December 1, 2021 15:41
@sujeong000
fix: test build 에러 해결
cbce413
@sujeong000
fix: 승차 알람 섹션 헤더가 보이지 않는 문제 해결
128ab18
fix: #343 해결 완료
8138817
feat: #345 마을버스, 다른 버스들 대응
03585b9
@Modyhoon
Merge branch 'develop' into test/RequestFactoryTests
2e38048
@Modyhoon
Merge pull request #330 from boostcampwm-2021/test/RequestFactoryTests
a27e93f 
[테스트] RequestFactoryTests 작성
@Modyhoon
Merge branch 'develop' into test/BusRouteViewModelTests
230e575
@Modyhoon
Merge pull request #334 from boostcampwm-2021/test/BusRouteViewModelT…
5b6cec2 
…ests

[테스트] BusRouteViewModelTests 작성
@Modyhoon
Merge pull request #335 from boostcampwm-2021/test/MovingStatusViewMo…
04f6780 
…delTests

[테스트] MovingStatusViewModelTests 작성
@Modyhoon
Merge pull request #336 from boostcampwm-2021/test/PersistentStorage
9587790 
[테스트] PersistenceStorage 작성 완료
@Modyhoon
Merge branch 'develop' into test/TokenManagerTests
94aa4cc
@Modyhoon
Merge pull request #337 from boostcampwm-2021/test/TokenManagerTests
6990426 
[테스트] TokenManager 테스트
@Modyhoon
Merge pull request #338 from boostcampwm-2021/test/StationViewModelTests
bab9d43 
[테스트] StationViewModel 테스트
@Modyhoon
Merge pull request #339 from boostcampwm-2021/test/SearchViewModelTests
3eb9bf4 
[테스트] SearchViewModel 테스트
@Modyhoon
Merge pull request #340 from boostcampwm-2021/test/AlarmSettingViewMo…
2e50150 
…delTests

[테스트] AlarmSettingViewModel 테스트
@Modyhoon
Merge branch 'develop' into test/NetworkServiceTests
739eccf
@Modyhoon
Merge pull request #341 from boostcampwm-2021/test/NetworkServiceTests
08ea36d 
[테스트] NetworkService 테스트
@Modyhoon
Merge pull request #342 from boostcampwm-2021/alarmSettingLayoutBugFix
3967764 
[버그] 승차 알람 섹션 헤더가 보이지 않는 문제 해결
@Modyhoon
Merge branch 'develop' into issue/#343
44d03e0
@Modyhoon
Merge pull request #344 from boostcampwm-2021/issue/#343
ece01b7 
[버그] 정거장 화면 진입시 특정 정거장에서 앱이 죽는 현상 해결
@Modyhoon
Merge pull request #346 from boostcampwm-2021/issue/#345
b8d794d 
feat: #345 마을버스, 다른 버스들 대응
feat: #343 결과 없으면 얼럿 창 띄우기
8882599
@tmfrlrkvlek tmfrlrkvlek added the 📲deployment 주별 배포 label Dec 2, 2021
@tmfrlrkvlek tmfrlrkvlek merged commit 7fd7b8a into main Dec 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@tmfrlrkvlek tmfrlrkvlek

@minsangKang minsangKang

@sujeong000 sujeong000

@Modyhoon Modyhoon

Labels
📲deployment 주별 배포
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@tmfrlrkvlek @minsangKang @sujeong000 @Modyhoon