Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
* feat: DomainCommon, FeatureCommon 모듈 생성 - #278 * refactor: 공통 타입들 DomainCommon, FeatureCommon으로 책임 분리 - #278 * refactor: common 분리에 영향받는 파일들 수정 - #278 * refactor: SharedDesignSystem에 있던 임시 TXItem 구현체 Feature에 맞게 분리 - #278 * refactor: TXItem 구현체 Feature분리에 따른 영향 범위 수정 - #278 * refactor: DomainStats common 의존성 제거 - #278 * refactor: RepeatCycle+Text재활용 리뷰 반영 - #278 --------- Co-authored-by: jihun <jihun332@gmai.com>
* feat: DomainCommon, FeatureCommon 모듈 생성 - #278 * refactor: 공통 타입들 DomainCommon, FeatureCommon으로 책임 분리 - #278 * refactor: common 분리에 영향받는 파일들 수정 - #278 * refactor: SharedDesignSystem에 있던 임시 TXItem 구현체 Feature에 맞게 분리 - #278 * refactor: TXItem 구현체 Feature분리에 따른 영향 범위 수정 - #278 * feat: CoreAnalytics 모듈 추가 - #281 * feat: AnalyticsClient, Event 구현 - #281 - draft로 올릴 예정이라 Event는 추후 추가 예정 * feat: 유져 로그인/로그아웃 시 Analytics user setting 하도록 구현 - #281 * feat: release일 때 analytics 동작하도록 구현 - #281 * refactor: CoreAnalytics 구조 개선 - #281 - CoreAnalytics에서 Feature책임을 갖고있던 기존 구조에서 Evetn를 protocol로 추상화 - logEvent 클로저로 통합 * feat: Auth 이벤트 심기 - #281 * feat: Onboarding 이벤트 심기 - #281 * feat: Home 이벤트 심기 - #281 * feat: MainTab 이벤트 심기 - #281 * feat: MakeGoal 이벤트 심기 - #281 * feat: GoalDetail 이벤트 심기 - #281 * feat: ProofPhoto 이벤트 심기 - #281 * feat: Stats 이벤트 심기 - #281 * docs: 누락된 주석 추가 - #281 * fix: 리뷰 반영 - #284 * feat: 추가 요청사항 인증샷 이벤트 심기 - #281 * fix: home 목표 생성 선택 이벤트 네이밍 수정 - #281 --------- Co-authored-by: jihun <jihun332@gmai.com>
* chore: FirebaseCrashlytics SPM 의존성 및 모듈 등록 - #286 * feat: Crashlytics dSYM 업로드 스크립트 추가 - #286 * feat: Core/Crashlytics 모듈 추가 - #286 * feat: Core 에러 타입에 CustomNSError 적용 - #286 * feat: Domain 에러 타입에 CustomNSError 적용 및 AuthClient 에러 wrapping - #286 * chore: App 타겟에 Crashlytics 의존성 및 dSYM 스크립트 추가 - #286 * feat: AppDelegate에 Crashlytics 수집 설정 추가 - #286 * feat: AppCoordinator에 Crashlytics 유저 식별자 및 오류 추적 추가 - #286 * feat: Auth Feature에 Crashlytics 로그인 실패 추적 추가 - #286 * feat: ProofPhoto Feature에 Crashlytics 오류 추적 추가 - #286 * feat: Onboarding Feature에 초대 코드 실패 토스트 및 Crashlytics 추적 추가 - #286 * chore: 버전 1.1.2로 업데이트 - #286 * fix: 컴파일 에러 수정 - #286 * fix: CalendarNow 변수명 SwiftLint identifier_name 위반 수정 - #286 * refactor: CrashlyticsClient를 화면별 이벤트 enum 패턴으로 전환 - #286 * refactor: AuthLoginError.caseName 제거 - #286
- CoreAnalytics에서 Feature책임을 갖고있던 기존 구조에서 Evetn를 protocol로 추상화 - logEvent 클로저로 통합
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
🔗 관련 이슈
📙 작업 내역