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

[Telemetry] Add telemetry to record how the app was opened on cold/warm/hot startup(home screen, applink, custom tab) #11830

Closed
7 tasks
sraturi opened this issue Jun 22, 2020 · 1 comment · Fixed by #11940 or #12114

Comments

@sraturi
Copy link
Contributor

sraturi commented Jun 22, 2020

Description & Product Manager / Data Scientist User Story

We'd like to have this telemetry so we're able to create performance tests that are representative of user experiences.

What questions will you answer with this data?

How do users open the app most frequently?

Acceptance Criteria

  • ENG files a DS JIRA request outlining their methodology.
  • DS sign off on instrumentation methodology addressing product questions.
  • Event pings can be queried via re:dash
  • Event pings can be queried via amplitude
  • We are sending telemetry events for the actions listed in the requirements
  • We have documented the telemetry
  • We have asked a data steward to review the telemetry

┆Issue is synchronized with this Jira Task

@github-actions github-actions bot added the needs:triage Issue needs triage label Jun 22, 2020
@sraturi sraturi added this to Needs prioritization in Performance, front-end roadmap via automation Jun 22, 2020
@sraturi sraturi changed the title [Telemetry] Add telemetry to record how the app was opened on warm/hot startup(home screen, applink, custom tab) [Telemetry] Add telemetry to record how the app was opened on cold/warm/hot startup(home screen, applink, custom tab) Jun 23, 2020
@sraturi sraturi moved this from Needs prioritization to In progress in Performance, front-end roadmap Jun 23, 2020
sraturi added a commit to sraturi/fenix that referenced this issue Jun 25, 2020
sraturi added a commit to sraturi/fenix that referenced this issue Jun 25, 2020
for mozilla-mobile#11830 collecting startup metric inaside onNewIntent in HomeActivity

clean up, added comments
@sraturi sraturi self-assigned this Jun 25, 2020
sraturi added a commit to sraturi/fenix that referenced this issue Jun 26, 2020
for mozilla-mobile#11830 collecting startup metric inaside onNewIntent in HomeActivity

clean up, added comments
sraturi added a commit to sraturi/fenix that referenced this issue Jun 26, 2020
for mozilla-mobile#11830 collecting startup metric inaside onNewIntent in HomeActivity

clean up, added comments
sraturi added a commit to sraturi/fenix that referenced this issue Jun 26, 2020
for mozilla-mobile#11830 collecting startup metric inaside onNewIntent in HomeActivity

clean up, added comments

nitpicks cleanup
sraturi added a commit to sraturi/fenix that referenced this issue Jun 26, 2020
for mozilla-mobile#11830 collecting startup metric inaside onNewIntent in HomeActivity

clean up, added comments

nitpicks cleanup

move all source startup telemetry into its own logic and added an UNKOWN state

nit cleanup

minor changes
sraturi added a commit to sraturi/fenix that referenced this issue Jun 26, 2020
for mozilla-mobile#11830 collecting startup metric inaside onNewIntent in HomeActivity

clean up, added comments

nitpicks cleanup

move all source startup telemetry into its own logic and added an UNKOWN state

nit cleanup

minor changes
sraturi added a commit to sraturi/fenix that referenced this issue Jun 27, 2020
for mozilla-mobile#11830 collecting startup metric inaside onNewIntent in HomeActivity

clean up, added comments

nitpicks cleanup

move all source startup telemetry into its own logic and added an UNKOWN state

nit cleanup

minor changes

added comment metric description

nit cleanup
Performance, front-end roadmap automation moved this from In progress to Done Jun 27, 2020
sraturi added a commit that referenced this issue Jun 27, 2020
…artup phases (#11940)

* for #11830 added new metric for collecting startup method

move all source startup telemetry into its own logic and added an UNKOWN state

* switched back to onNewIntent solution

* renamed the metric
@liuche liuche mentioned this issue Jun 27, 2020
12 tasks
@sraturi sraturi moved this from Done to In progress in Performance, front-end roadmap Jun 29, 2020
@sraturi
Copy link
Contributor Author

sraturi commented Jun 29, 2020

Reopening issue again to find a proper solution

@sraturi sraturi reopened this Jun 29, 2020
sraturi added a commit to sraturi/fenix that referenced this issue Jun 29, 2020
…ng AllStartup telemetry logic

lint check
@sraturi sraturi linked a pull request Jun 30, 2020 that will close this issue
4 tasks
sraturi added a commit to sraturi/fenix that referenced this issue Jun 30, 2020
…ng AllStartup telemetry logic

lint check
sraturi added a commit to sraturi/fenix that referenced this issue Jul 2, 2020
…ng AllStartup telemetry logic

lint check
sraturi added a commit to sraturi/fenix that referenced this issue Jul 9, 2020
…ng AllStartup telemetry logic

lint check

renamed the intentReceived telemetry to appOpenedAllSource

added comments

removed unused code

moved lifecycle process to AppAllSourceStartTelemetry

moved tracking event out of init function

lint fix
sraturi added a commit to sraturi/fenix that referenced this issue Jul 11, 2020
…ng AllStartup telemetry logic

lint check

renamed the intentReceived telemetry to appOpenedAllSource

added comments

removed unused code

moved lifecycle process to AppAllSourceStartTelemetry

moved tracking event out of init function

lint fix

moved appAllStartTelemetry to components

added bit more info about the metrics

added the  onReceivedIntent metric back

minor fix
sraturi added a commit to sraturi/fenix that referenced this issue Jul 23, 2020
…ng AllStartup telemetry logic

lint check

renamed the intentReceived telemetry to appOpenedAllSource

added comments

removed unused code

moved lifecycle process to AppAllSourceStartTelemetry

moved tracking event out of init function

lint fix

moved appAllStartTelemetry to components

added bit more info about the metrics

added the  onReceivedIntent metric back

minor fix

change discriptions based on the comments frm MR

wrote test cases for AppAllSourceStartTelemetry.kt

lint fix
sraturi added a commit to sraturi/fenix that referenced this issue Jul 24, 2020
…ng AllStartup telemetry logic

lint check

renamed the intentReceived telemetry to appOpenedAllSource

added comments

removed unused code

moved lifecycle process to AppAllSourceStartTelemetry

moved tracking event out of init function

lint fix

moved appAllStartTelemetry to components

added bit more info about the metrics

added the  onReceivedIntent metric back

minor fix

change discriptions based on the comments frm MR

wrote test cases for AppAllSourceStartTelemetry.kt

lint fix

test case to mock application going background

post rebase:

post rebase:
sraturi added a commit to sraturi/fenix that referenced this issue Jul 25, 2020
…ng AllStartup telemetry logic

lint check

renamed the intentReceived telemetry to appOpenedAllSource

added comments

removed unused code

moved lifecycle process to AppAllSourceStartTelemetry

moved tracking event out of init function

lint fix

moved appAllStartTelemetry to components

added bit more info about the metrics

added the  onReceivedIntent metric back

minor fix

change discriptions based on the comments frm MR

wrote test cases for AppAllSourceStartTelemetry.kt

lint fix

test case to mock application going background

post rebase:

post rebase:

fixed nit from comments

fixed nit from comments

fixed nit from comments

lint fix

lint fix
Performance, front-end roadmap automation moved this from In progress to Done Jul 29, 2020
mcomella pushed a commit that referenced this issue Jul 29, 2020
…telemetry logic

lint check

renamed the intentReceived telemetry to appOpenedAllSource

added comments

removed unused code

moved lifecycle process to AppAllSourceStartTelemetry

moved tracking event out of init function

lint fix

moved appAllStartTelemetry to components

added bit more info about the metrics

added the  onReceivedIntent metric back

minor fix

change discriptions based on the comments frm MR

wrote test cases for AppAllSourceStartTelemetry.kt

lint fix

test case to mock application going background

post rebase:

post rebase:

fixed nit from comments

fixed nit from comments

fixed nit from comments

lint fix

lint fix
@data-sync-user data-sync-user changed the title [Telemetry] Add telemetry to record how the app was opened on cold/warm/hot startup(home screen, applink, custom tab) FNX3-14939 ⁃ [Telemetry] Add telemetry to record how the app was opened on cold/warm/hot startup(home screen, applink, custom tab) Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX3-14939 ⁃ [Telemetry] Add telemetry to record how the app was opened on cold/warm/hot startup(home screen, applink, custom tab) FNX-12815 ⁃ [Telemetry] Add telemetry to record how the app was opened on cold/warm/hot startup(home screen, applink, custom tab) Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX-12815 ⁃ [Telemetry] Add telemetry to record how the app was opened on cold/warm/hot startup(home screen, applink, custom tab) FNX2-13637 ⁃ [Telemetry] Add telemetry to record how the app was opened on cold/warm/hot startup(home screen, applink, custom tab) Aug 11, 2020
@kbrosnan kbrosnan changed the title FNX2-13637 ⁃ [Telemetry] Add telemetry to record how the app was opened on cold/warm/hot startup(home screen, applink, custom tab) [Telemetry] Add telemetry to record how the app was opened on cold/warm/hot startup(home screen, applink, custom tab) Aug 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.