Skip to content
This repository has been archived by the owner on Jan 29, 2025. It is now read-only.

Remove references to "release channel" in Audiences.ts now that channel is parameterized? #106

Open
pdehaan opened this issue Jul 28, 2020 · 1 comment

Comments

@pdehaan
Copy link
Contributor

pdehaan commented Jul 28, 2020

Looks like the audience descriptions still reference a "release channel" even though we turned that into a variable. Not sure if that is a bug or if it'd only impact us testing locally on Beta/Nightly and no end users would see this.

description: "All users in en-* locales using the release channel.",

description: "All users in the US with an en-* locale using the release channel.",

"First start-up users (e.g. for about:welcome) with an en-* locale using the release channel.",

  us_only: {
    name: "US users (en)",
    description: "All users in the US with an en-* locale using the release channel.",
    targeting:
      "localeLanguageCode == 'en' && region == 'US' && browserSettings.update.channel == '{firefox_channel}'",
    desktop_telemetry:
      "STARTS_WITH(environment.settings.locale, 'en') AND normalized_country_code = 'US'",
  },

Also, I notice that the targeting field references a {firefox_channel}, but I don't see a similar reference to a channel in the desktop_telemetry field. I'll need to peek further into that to see what it was like before we parameterized the channel from a hardcoded "release" string.

@pdehaan
Copy link
Contributor Author

pdehaan commented Jul 28, 2020

Also, I notice that the targeting field references a {firefox_channel}, but I don't see a similar reference to a channel in the desktop_telemetry field. I'll need to peek further into that to see what it was like before we parameterized the channel from a hardcoded "release" string.

Looks like this was changed in PR #99. I also added a more context aware question in #99 (comment)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant