Skip to content

feat: AdMobバナー広告の導入#148

Merged
stotic-dev merged 19 commits into
mainfrom
feat/setup-admob-sdk
May 16, 2026
Merged

feat: AdMobバナー広告の導入#148
stotic-dev merged 19 commits into
mainfrom
feat/setup-admob-sdk

Conversation

@stotic-dev
Copy link
Copy Markdown
Owner

@stotic-dev stotic-dev commented May 16, 2026

経緯

  • Firebase が従量課金制のため、ユーザー増加にともない運用コストが膨らむ問題があり、収益化スキームとしてバナー広告を導入する
  • 関連Issue: 広告の追加 #142

実装内容

AdMob SDK の導入

  • GoogleMobileAds を SPM 依存に追加し、MobileAdsClient を経由してアプリ起動時に初期化する
  • 起動処理を HometeApp に組み込み

広告ユニットIDの管理方針

  • 環境ごと(dev / 本番)で値を切り替えるため、ユニットIDは xcconfig → Info.plist 経由で取得する設計に
    • BannerType.unitId で Info.plist の AdUnitIdList 辞書から該当キーの値を取得
  • 本番ID を含む Admob.xcconfig はリポジトリ管理外.gitignore 済み)。Admob_dev.xcconfig はサンプルIDのみ含むためリポジトリ管理に残す
  • 理由: 本番のアプリID・広告ユニットIDは機密情報として扱い、誤コミットを防ぐため

依存関係の整理

  • AdComponentResolverHometeDomain 側に置き、HometeUI 経由で各 Feature から広告コンポーネントを参照できるように整理
  • 理由: View 層(Feature モジュール)が HometeInfrastructure / GoogleMobileAds に直接依存しないようにし、モジュール境界とテスト容易性を保つため

表示・テスト

  • HomeView のダッシュボード上部にバナー広告を配置(現状はテストモードのサンプルユニットID)
  • VRT スナップショットを新レイアウトに更新

確認内容

  • Dev ビルドを実行し、HomeView のダッシュボード上部にバナー広告が表示されること
  • 広告のロード失敗時にレイアウトが崩れないこと
  • リリースビルドが警告なく通ること
  • VRT (snapshotTesting)が通ること
  • Admob.xcconfig(本番ID)がリポジトリに含まれていないこと

@stotic-dev-bot
Copy link
Copy Markdown
Collaborator

stotic-dev-bot commented May 16, 2026

Warnings
⚠️ PRの変更行が多すぎます。500行以内にしてね!理想は400行!

snapshotの変更

更新ファイル: AppTabView_0-iPhone-16.1.png

before after
image image

更新ファイル: AppTabView_0-iPhone-SE-2nd-generation.1.png

before after
image image

Generated by 🚫 Danger Swift against e084a22

@stotic-dev stotic-dev self-assigned this May 16, 2026
@stotic-dev stotic-dev added the feature 機能追加 label May 16, 2026
@stotic-dev-bot
Copy link
Copy Markdown
Collaborator

stotic-dev-bot commented May 16, 2026

File Coverage
LocalPackage/Sources/HometeUI/ViewUtilities/Advertisement/AdComponentResolver.swift 0.0% ⚠️

Generated by 🚫 Danger Swift against e084a22

stotic-dev and others added 7 commits May 16, 2026 13:05
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Xcode CloudへCD移行に伴い、cd_release.yml/cd_testFlight.ymlは不要になったため削除する。

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
ci: Xcode CloudのCIスクリプトをワークフロー別に分岐させる
@stotic-dev stotic-dev merged commit bc530dd into main May 16, 2026
2 of 3 checks passed
@stotic-dev stotic-dev deleted the feat/setup-admob-sdk branch May 16, 2026 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature 機能追加

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants