Skip to content

Create Week10 Mission 1#47

Open
chldnjswo wants to merge 2 commits into
mainfrom
Week10/Deku
Open

Create Week10 Mission 1#47
chldnjswo wants to merge 2 commits into
mainfrom
Week10/Deku

Conversation

@chldnjswo
Copy link
Copy Markdown
Contributor

📝 미션 번호

10주차 Mission 1

📋 구현 사항

  • release Keystore를 연결하여 배포용 AAB 생성
  • ProfileViewModel의 기존 요청을 취소하고 로딩 상태를 즉시 반영하여 프로필 조회 중복 요청 및 이전 응답의 상태 덮어쓰기 방지

📎 스크린샷

✅ 체크리스트

  • Merge 하려는 브랜치가 올바르게 설정되어 있나요?
  • 에뮬레이터 또는 실제 기기에서 정상 동작하나요?
  • 불필요한 주석 및 Log가 제거되었나요?

🤔 질문 사항

@chldnjswo chldnjswo requested a review from Dawon-Y May 31, 2026 10:01
@chldnjswo chldnjswo self-assigned this May 31, 2026
@Dawon-Y
Copy link
Copy Markdown
Contributor

Dawon-Y commented Jun 1, 2026

안녕하세요! 10주차 과제 진행하시느라 고생 많으셨습니다💚


1) 배포용 AAB 빌드 (PR에 AAB 경로 기재)

빌드 명령

Week10/Deku 루트에서:

  • Windows:
.\gradlew.bat bundleRelease
  • macOS/Linux:
./gradlew bundleRelease

AAB 파일 경로 (PR 본문에 적을 것)

Gradle 기본 출력 경로는 동일합니다:

app/build/outputs/bundle/release/app-release.aab

이 경로를 PR에 그대로 “업로드(기재)”하면 1번 체크 완료입니다.


2) Claude Code로 코드 리뷰 받기 + 개선 반영 (지금 폴더 기준)

현재 확인된 것

  • app/build.gradle.kts에서
    • REQRES_API_KEYlocal.properties 또는 env에서 읽어서 BuildConfig로 넣음
    • keystore.properties가 있을 때만 release signingConfig를 생성/적용
      배포용/보안 관점 기본기 좋음

하지만 과제 제출 관점에서 “부족한 것”

  • Week10/LinLin에는 CLAUDE.md, RELEASE.md가 있었고
    Week10/Yido에는 README.md에 체크리스트가 있었는데,
  • Week10/Deku 폴더 루트에는 그런 증빙 문서가 없음 (현재 listing 기준)

과제 요구사항이 “CLI 설치/실행/리뷰 요청/개선”을 체크하는 형태라,
최소한 README.md 또는 RELEASE.md 하나는 있어야 좋습니다.


3) 참고: Deku의 build.gradle에서 좋은 점/주의점

좋은 점

  • 키 주입:
val reqResApiKey = localProperties.getProperty("REQRES_API_KEY")
    ?: System.getenv("REQRES_API_KEY")
    ?: ""
buildConfigField("String", "REQRES_API_KEY", buildConfigString(reqResApiKey))

→ 로컬/CI 모두 대응 가능.

  • release signingConfig도 파일 존재 조건으로 안전하게 처리:
if (keystorePropertiesFile.exists()) { create("release") { ... } }
...
if (keystorePropertiesFile.exists()) { signingConfig = signingConfigs.getByName("release") }

주의(제출 안정성)

  • REQRES_API_KEY가 비어 있어도 빌드 자체는 되는데(빈 문자열), 런타임 API 호출이 실패할 수 있음.
    • 과제 기준으로는 “키 설정 방법”을 문서에 명시해주는 편이 좋습니다.

워크북 진행하시느라 수고하셨습니다! 궁금한 점 있으시면 언제든 말씀해 주세요!

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.

2 participants