Skip to content

refactor(setting): 계정·서비스 알림·마케팅 동의 8멤버를 설정 모듈로 이동 #1429

Closed

Description

@1hyok

작업 유형

refactor — 동작 변경 없는 구조·책임·경계 개선

주 담당 모듈

setting — 설정

개요

설정만 소비하는 계정·연결 계정·서비스 알림·마케팅 동의의 8멤버를 PR #2123에서 feature:setting으로 직접 이동한다. core에 임시 계약을 만들거나 후속 PR에서 같은 VM을 다시 옮기지 않는다.

구현 범위

  • setting domain의 SettingAccountRepository는 탈퇴/연결 조회/연결/해제를, SettingNotificationRepository는 서비스 알림 조회/수정과 마케팅 동의 조회/수정을 소유한다.
  • setting data가 구현·DTO 매퍼·푸시 실패 매핑·Hilt 바인딩을 소유하고 기존 외부 동작을 보존한다.
  • core의 합본 계약·구현·fake에서 8멤버를 제거했다. 수신자·프로필을 사용하는 다른 모듈의 호환 계약은 유지한다.
  • 앱 Hilt 대체 모듈과 관련 테스트 fixture도 최종 소유 모듈에 맞췄다.
  • core data 71개, setting data 16개 테스트와 앱 main/unit/androidTest 컴파일을 검증했다. 탈퇴 후 세션 정리와 푸시 실패 매핑 회귀를 새 소유 모듈에서 검증한다.

병합 및 의존 순서

#1429와 #1739의 저장소 책임 분리와 실제 소비를 develop 기준 PR #2123에 함께 담았다. 닫힌 #1967에서는 이 범위만 추출했고 MVI·Nav3 구현은 복구하지 않았다. 이 이슈는 #2123 머지로 종료한다. 로컬 검증과 최신 CI가 통과했다. API 30 계측 테스트 57개도 실패 없이 통과했으며, 필수 리뷰와 병합을 기다린다.

Activity

  1. added
    refactor동작 변경 없이 코드 구조·책임·경계를 개선하는 작업
    on Aug 28, 2026
  2. assigned and unassigned on Sep 6, 2026
  3. changed the title [-]refactor(setting): 계정·푸시 설정 6멤버를 core 경유 없이 feature:setting 으로 하향 (#1282 갈래)[/-] [+]refactor(setting): 계정·서비스 알림·마케팅 동의 8멤버를 설정 모듈로 이동[/+] on Sep 12, 2026
  4. added
    pr-open이 이슈를 구현하는 PR 이 열려 있다 — base 가 기본 브랜치가 아니어도 드러낸다
    on Sep 13, 2026
  5. 1hyok commented on Sep 13, 2026

    @1hyok
    CollaboratorAuthor

    이 이슈를 맡은 PR 은 #1967 이다 (base codex/1695-setting-nav3). 본문의 "확정된 저장·소유 계약 통합" 절이 계정 삭제·연결을 SettingAccountRepository 로, 서비스 알림·마케팅 동의를 SettingNotificationRepository 로 옮기고 core 합본 계약·구현·fake 에서 그 8멤버를 걷어낸다. 앱 Hilt 대체 모듈도 함께 옮겼다.

    pr-open 라벨을 손으로 붙였다. 라벨러가 PR 제목의 대표 이슈 하나만 읽어서, 본문에 Closes 가 여럿인 PR 의 두 번째부터를 놓치기 때문이다(#2082). #1967 은 제목이 (#1502) 로 끝나고 본문에서 #1502 · #1691 · #1739 · 이 이슈 넷을 닫는데, 라벨은 #1502 하나만 받았다.

    base 가 기본 브랜치가 아니라서 GitHub 도 이 이슈의 Development 칸에 아무것도 표시하지 않는다. 그래서 이 이슈에는 신호가 하나도 없었고, 미착수로 읽힐 자리였다.

    이 코멘트에는 라벨러가 쓰는 마커를 그대로 넣어 두었다. #2082 수정이 머지된 뒤 리컨사일러가 같은 말을 다시 달지 않는다. 다만 라벨 쪽은 수정이 머지되기 전까지 배포본 리컨사일러가 스테일로 보고 도로 떼어 간다. 라벨이 안 보이면 그건 #2082 가 아직 안 고쳐진 것이고, 착수 여부의 정본은 이 코멘트다.

  6. added
    pr-open이 이슈를 구현하는 PR 이 열려 있다 — base 가 기본 브랜치가 아니어도 드러낸다
    and removed
    pr-open이 이슈를 구현하는 PR 이 열려 있다 — base 가 기본 브랜치가 아니어도 드러낸다
    on Sep 13, 2026
  7. removed
    pr-open이 이슈를 구현하는 PR 이 열려 있다 — base 가 기본 브랜치가 아니어도 드러낸다
    on Sep 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

area:setting설정 기능 영역refactor동작 변경 없이 코드 구조·책임·경계를 개선하는 작업

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions