Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Commit

Permalink
For #10426: Removes installation ping GAID (#10532)
Browse files Browse the repository at this point in the history
  • Loading branch information
sblatz committed May 9, 2020
1 parent d58e7d3 commit 17fb7c1
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 18 deletions.
13 changes: 0 additions & 13 deletions app/metrics.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2073,19 +2073,6 @@ installation:
notification_emails:
- fenix-core@mozilla.com
expires: "2020-09-01"
identifier:
send_in_pings:
- installation
type: string
description: |
The hashed and salted GAID. Used for a short term installation validation test.
bugs:
- https://github.com/mozilla-mobile/fenix/issues/10426
data_reviews:
- https://github.com/mozilla-mobile/fenix/pull/10446#issuecomment-624816258
notification_emails:
- fenix-core@mozilla.com
expires: "2020-05-10"

browser.search:
with_ads:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -64,10 +64,6 @@ class InstallationPing(private val context: Context) {
}

CoroutineScope(Dispatchers.IO).launch {
MetricsUtils.getHashedIdentifier(context)?.let {
Installation.identifier.set(it)
}

Pings.installation.submit()
markAsTriggered()
}
Expand Down
1 change: 0 additions & 1 deletion docs/metrics.md
Original file line number Diff line number Diff line change
Expand Up @@ -223,7 +223,6 @@ The following metrics are added to the ping:
| installation.adgroup |[string](https://mozilla.github.io/glean/book/user/metrics/string.html) |The name of the AdGroup that was used to source this installation. |[1](https://github.com/mozilla-mobile/fenix/pull/8074#issuecomment-586480836)||2020-09-01 |
| installation.campaign |[string](https://mozilla.github.io/glean/book/user/metrics/string.html) |The name of the campaign that is responsible for this installation. |[1](https://github.com/mozilla-mobile/fenix/pull/8074#issuecomment-586512202)||2020-09-01 |
| installation.creative |[string](https://mozilla.github.io/glean/book/user/metrics/string.html) |The identifier of the creative material that the user interacted with. |[1](https://github.com/mozilla-mobile/fenix/pull/8074#issuecomment-586512202)||2020-09-01 |
| installation.identifier |[string](https://mozilla.github.io/glean/book/user/metrics/string.html) |The hashed and salted GAID. Used for a short term installation validation test. |[1](https://github.com/mozilla-mobile/fenix/pull/10446#issuecomment-624816258)||2020-05-10 |
| installation.network |[string](https://mozilla.github.io/glean/book/user/metrics/string.html) |The name of the Network that sourced this installation. |[1](https://github.com/mozilla-mobile/fenix/pull/8074#issuecomment-586512202)||2020-09-01 |
| installation.timestamp |[datetime](https://mozilla.github.io/glean/book/user/metrics/datetime.html) |The date and time of the installation. |[1](https://github.com/mozilla-mobile/fenix/pull/8074#issuecomment-586512202)||2020-09-01 |

Expand Down

0 comments on commit 17fb7c1

Please sign in to comment.