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.

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

Feature/product logging #67

Merged
merged 17 commits into from
Sep 18, 2024
Merged

Feature/product logging #67

merged 17 commits into from
Sep 18, 2024

Conversation

J0onYEong
Copy link
Contributor

@J0onYEong J0onYEong commented Sep 18, 2024

변경된점

  • Amplitude로깅 추가
  • 단계벌 작업이 필요한 기능에 대해 변경된 UI반영

Amplitude로깅추가

특정 기능이 로깅시스템을 사용하돼 의존성 주입을 위해 각 Feature별 Logger인터페이스를 만들었습니다.
로깅시스템이 복잡하지 않다고 판단하여 하나의 타입(OverallLogger)에서 모든 로깅로직 프로토콜의 구체타입을 구현하도록 했습니다.
image

스크린샷 2024-09-18 오후 2 11 20

DefaultOverallLogger는 실제로 이벤트를 전송하는 객체(publishr)를 프로퍼티로 가집니다.

스크린샷 2024-09-18 오후 2 12 01

해당 객체를 의존성 주입을 통해 콘솔 출력용 Debug용 객체와 Amplitude를 사용한 실제 로깅 객체를 주입가능하도록 구현했습니다.

스크린샷 2024-09-18 오후 2 12 24

각각의 로커 interface에는 동일한 객체가 주입되는 구조이며, 해당 객체(overallLogger)는 DI컨테이너에 의해 단 하나의 인스턴스만 생성됩니다.

스크린샷 2024-09-18 오후 2 14 05

Ampltude에 로깅이 잘 찍히는 것을 확인할 수 있었습니다.

스크린샷 2024-09-18 오후 2 17 49

변경된 UI반영

  • 상단 네비게이션의 뒤로가기 버튼을 누를 경우 화면을 이탈하도록 변경했습니다.
  • 하단 버튼에 뒤로가기 버튼을 추가했습니다.

@J0onYEong J0onYEong merged commit abc9d0a into develop Sep 18, 2024
1 check failed
@J0onYEong J0onYEong deleted the feature/product_logging branch September 18, 2024 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant