feat: BibbiNetworkMonitor Legacy 코드 제거 #681
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🔵PR을 올리기 전 아래 사항을 확인해주세요.
(위 내용은 지워주세요)
😽개요
BibbiNetworkMonitor
Legacy코드 제거 및BBNetworkEventMonitor
로 EventMonitor 적용BibbiNetworkMonitor
내부 BBLogger 코드 적용BBLogger
모듈 내부 LogLevel,
Log Method,
Log ObservableType` 추가app_store_build_number
메서드 추가Marketing Version
1.2.3으로 수정 및UILaunchStoryboardName
확장자 제거🛠️작업 내용
BBLogger Module 구현
Logger
Struct를 활용해서 BBLogger를 구현했습니다. Logger 같은 경우MetaDataType
에 따라 출력된 데이터를 확인할 수 있으며 Category는 해당 메서드 파라미터에 지정하도록 구현했습니다.Category
를Nested type
으로 빼두지 않은 이유는 유동적이어야 하기 때문입니다. Logger의 장점은 필터링을 통해 원하는 출력값을 빠르게 조회할 수 있다는 점입니다. 하지만 Category를 임의로 지정을 해두면 한정적인 자원에서 필터링을 할 수밖에 없기 때문에 조회 가정에서 많은 어려움이 있습니다. 추가로 여러case
가 추가될 수도 있기 때문입니다.app_store_build_number
추가했습니다. 추가한 이유는 중간에 Xcode로 테플에 업로드 시 github Action으로 가져오는 BuildNumber와 매칭이 안되기 때문에 이미 테플에 올라가있는 BuildNumber를 활용해 증가시키도록 수정하였습니다.BBLogger Module 출력 내용
Method Name
,FileName
,timestamp
을 출력하도록 구현했습니다.|
✅테스트 케이스
BibbiNetworkMonitor
에 로그가 출력되는지 확인해요🙏🏻아래와 같이 PR을 리뷰해주세요.