YAPP-Github / 24th-App-Team-3-iOS

6 stars 0 forks source link

[Feature] #20 Logger를 활용한 Common 로그기능 개발 #23

Closed dlrjswns closed 3 months ago

dlrjswns commented 3 months ago

Describe

Logger인스턴스를 활용하여 debug, info, notice, error 레벨에 해당하는 API를 쉽게 사용할 수 있도록 추상화 작업을 진행

Works made

Changes Made

As-Is

기존 로직

To-BE

변경 로직

스크린샷

Console 앱 로그(하위시스템을 이용해 필터링하여 로그 보기)

스크린샷 2024-06-13 오전 12 59 16

Xcode Console 로그

스크린샷 2024-06-13 오전 12 45 51

How to Test

코드 사용법

LogHelper.debug("LogHelper Debug API 호출", privacy: .public)
LogHelper.notice("LogHelper notice API 호출", privacy: .public)
LogHelper.error("LogHelper error API 호출", privacy: .public)
LogHelper.info("LogHelper info API 호출", privacy: .public)
스크린샷 2024-06-13 오전 1 11 00

Issues Resolved

Additional context

References

https://hereismyblog.tistory.com/31

dlrjswns commented 3 months ago
[ShowPot/BaseViewController.swift:46] setupStyles() viewDidLoad() 호출

@ChoiysApple 지금 BaseViewController에 로그를 찍으니까 위에처럼 찍히는데 저는 LoginViewController가 찍히게끔 하고싶은데 그럴려면 LoginViewController에서 로그를 찍어야할거같은데 그건 좀 비효율적일까요 ??

dlrjswns commented 3 months ago

지금 develop 커밋들이 feature/#20-logger-dev 브랜치에만 반영되어있어서, 제가 viewConvention 브랜치에 작업한 내용들도 해당 PR에서 보여지고 있습니다.

타겟 브랜치를 develop으로 바꾸시거나, feature/#20-logger-master도 rebase 완료해서 다시 PR 올려주세요

아 넵 !! 타겟 브랜치를 logger-master -> develop 브랜치로 변경했습니다 !! 감사합니다 👍👍👍