Inconsistent docs/validation for Event.event_source in Google Ads offline conversion migration
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 45/100
- issue の種類
- ドキュメント
- 明瞭さ
- おおむね明確
- 活発さ
- 静か
- 技術スタック
- php
- 領域
- api, documentation
調査の方向性
Google Ads オフライン コンバージョンのフィールドマッピング表と、issue に記載されている IngestEvents/events:ingest の動作から始め、次に Event.event_source の生成された PHP protobuf フィールドドキュメントを確認します。必要な動作と conversion_environment との推奨される関係を確認し、期待されるリクエストフィールドとマッピングが明確になるよう、移行ドキュメントと API リファレンスを更新します。
索引モデルが issue の本文から書いたものです。
説明
Hi,
I noticed an inconsistency while migrating Google Ads offline conversion uploads to the Data Manager API.
In the Google Ads offline conversion upgrade field mappings:
the table says:
ClickConversion.conversion_environment -> No equivalent
No equivalent -> Event.event_source
This suggests that conversion_environment should not be mapped directly to event_source, and that event_source is a separate Data Manager field.
However, when calling IngestEvents / events:ingest without event_source, the API rejects the request with:
{
"field": "events.events[0].event_source",
"description": "Required field is missing.",
"reason": "REQUIRED_FIELD_MISSING"
}
This makes event_source effectively required for Google Ads event ingestion, even though it is shown as having no Google Ads API equivalent in the migration table, and the generated PHP protobuf class documents it as optional.
Could the documentation clarify the recommended migration behavior?
Specifically:
- Is event_source required for Google Ads offline conversion event ingestion?
- If yes, should advertisers set it explicitly based on their own source context, e.g. WEB for website purchases?
- Should conversion_environment = WEB|APP be intentionally mapped to event_source = WEB|APP, despite the mapping table saying conversion_environment has no equivalent?
- Should the API reference/protobuf field behavior mark event_source as required for this use case?
Current workaround:
We keep conversion_environment for Google Ads API uploads and add a separate explicit Data Manager field:
{
"conversion_environment": "WEB",
"event_source": "WEB"
}
Then the Data Manager adapter maps only event_source to Event.event_source, avoiding an undocumented automatic mapping from conversion_environment.
- 主要言語
- PHP
- スター
- 1
- フォーク
- 1
- 平均マージ
- 22時間 23分
- マージ済み PR（30日）
- 5
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
googleads/data-manager-php のほかの issue
-
Dependency Dashboardオープンtype: process
難易度 1/5 1時間未満 初心者へのやさしさ 20/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 52/100
googleads/data-manager-php の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
codeigniter4/CodeIgniter4#10616 ·
メンテナーはふだん 1 日以内に返信
-
bug code quality
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
SemanticMediaWiki/SemanticMediaWiki#7149 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
-
bug
難易度 1/5 1時間未満 初心者へのやさしさ 82/100
FriendsOfFlarum/upload#527 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
google/site-kit-wp#13825 ·
メンテナーはふだん 3 日以内に返信