Skip to content

Conversation

@s9hn
Copy link
Member

@s9hn s9hn commented Mar 23, 2025

📌𝘐𝘴𝘴𝘶𝘦𝘴

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

  • :core:resource 모듈이 생성되었습니다.

    • :app:debug[main] 패키지를 :core:resource:debug[main]으로 옮겼습니다.
    • :app:res 패키지를 :core:resource:res으로 옮겼습니다.
  • :app:res 패키지엔 다음과 같은 항목들을 포함하고 있습니다.

    • xml 뷰 시스템 종속적인 리소스들(typography.xml, color.xml, selector, shape, layer-list)
    • xml 뷰 시스템 설정 파일들(theme.xml, styles.xml)
    • 레이아웃 파일들(menu, item, fragment, activity)
  • :core:resource:res 패키지엔 다음과 같은 항목들을 포함하고 있습니다.

    • 특정 뷰 시스템에 종속적이지 않은 순수 리소스 파일
    • raw(json파일)
    • drawable, drawable-#dpi(png, svg, gif)
    • font(otf, ttf)
    • mipmap-#dpi
    • string 상수들(deeplinks, strings)

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

image

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

  • 멀티 모듈 환경을 대비해, 특정 뷰 시스템에 종속되지 않는 순수 리소스 파일들을 리소스 모듈로 일원화합니다.

    • xml과 compose 두 환경에서 같은 리소스 파일을 재사용할 수 있습니다.
    • 추후 작업 시, 뷰(스크린) 별로 생길 수 있는 중복 리소스 파일을 제거할 수 있습니다.
    • 한번의 리소스 모듈 빌드로 별도의 리소스 추가 빌드를 진행하지 않아도 됩니다.
  • 앱 모듈엔 기존 xml 뷰 시스템에 종속적인 xml 파일들만 존재합니다.

  • 따라서 compose로 마이그레이션 및 모듈화 작업이 QA 및 배포가 되면, 기존 앱모듈의 리소스 파일 및 패키지는 제거해주셔도 됩니다.

  • 기존처럼 R 클래스를 임포트해 사용할 수 있습니다.

앞으로 리소스 사용 시, 리소스 모듈에서 먼저 찾아주세요. 가급적이면 중복 리소스를 방지하기 위해 모두 노력해주세요.

@s9hn s9hn added 🍯 [FEAT] 새로운 기능을 개발합니다. ♻️ [REFACTOR] 기존 코드를 리팩토링합니다. ⚖️ 재상 세훈 전지적 세훈 시점 🚧 [BUILD] 빌드 업무 수정, 패키지 매니저 수정합니다. labels Mar 23, 2025
@s9hn s9hn self-assigned this Mar 23, 2025
@s9hn s9hn changed the title Feat/613 refactor: 프로젝트 Modularization 적용 (1) Mar 23, 2025
@s9hn s9hn requested review from junseo511, m6z1 and yeonjeen and removed request for junseo511 and yeonjeen March 23, 2025 07:30
@yeonjeen yeonjeen requested review from a team and removed request for junseo511 and m6z1 March 23, 2025 07:30
@s9hn s9hn requested a review from m6z1 March 23, 2025 07:30
@s9hn s9hn requested review from junseo511 and yeonjeen March 23, 2025 07:30
Copy link
Member

@m6z1 m6z1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨어요!

Copy link
Member

@yeonjeen yeonjeen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다~

Copy link
Member

@junseo511 junseo511 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

진짜 완전 노가다였을텐데 고생 많으셨습니다 👍

@s9hn s9hn merged commit 7185531 into feat/612 Mar 24, 2025
@m6z1 m6z1 deleted the feat/613 branch March 24, 2025 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚖️ 재상 세훈 전지적 세훈 시점 🚧 [BUILD] 빌드 업무 수정, 패키지 매니저 수정합니다. 🍯 [FEAT] 새로운 기능을 개발합니다. ♻️ [REFACTOR] 기존 코드를 리팩토링합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants