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

Add new Leanplum E_Changed_Default_To_Fenix Event #12103

Closed
5 tasks
cbonacuse opened this issue Jun 29, 2020 · 15 comments
Closed
5 tasks

Add new Leanplum E_Changed_Default_To_Fenix Event #12103

cbonacuse opened this issue Jun 29, 2020 · 15 comments
Assignees
Labels
🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified needs:triage Issue needs triage P1 Current sprint

Comments

@cbonacuse
Copy link

cbonacuse commented Jun 29, 2020

The following were identified as missing in Fenix during a recent test by the Mobile Engagement team. This event by a similar name (E_Changed_Default_To_Fennec) is present in Fennec. This is requested to be in market by 8/25/20 (release 80).

  • E_Changed_Default_To_Fenix

This should be fired when the user changes their device setting to have the default browser sent to Firefox (Fenix), regardless of the app being presently opened or not.

Tasks:

┆Issue is synchronized with this Jira Task

@cbonacuse cbonacuse added the 🐞 bug Crashes, Something isn't working, .. label Jun 29, 2020
@cbonacuse cbonacuse added P1 Current sprint Leanplum labels Jun 29, 2020
@github-actions github-actions bot added the needs:triage Issue needs triage label Jun 29, 2020
@sblatz sblatz self-assigned this Jul 27, 2020
@ekager ekager added the eng:qa:needed QA Needed label Jul 29, 2020
@ekager
Copy link
Contributor

ekager commented Jul 29, 2020

cc @cbonacuse this has landed!

@AndiAJ
Copy link
Collaborator

AndiAJ commented Jul 30, 2020

Hi, verified as fixed on both the latest Nightly Build 200730 and Fenix Beta 80.0.0-beta-1 using the following devices:
• Google Pixel 3a (Android 10)
• Huawei Mate 20 Lite (Android 9)
• OnePlus A3 (Android 6.0.1)

✔️ Nightly

Google Pixel 3a - device ID e6222324-6a2d-403a-abc5-a7b777227632

image

Huawei Mate 20 Lite - device ID 1334424f-5898-4337-9ed7-9679fed9fb25

image

OnePlus A3 - device ID d5e35e4f-c75a-4b5e-9aa0-6aa2cbb6c78b

image

✔️ Beta

Google Pixel 3a - device ID 35d5c40d-0362-4b55-8aee-ffc6a9e7012c

image

Huawei Mate 20 Lite - device ID 48e981ae-5ec4-43f7-bbb2-5d4532f15627

image

OnePlus A3 - device ID 27951d4c-5729-43cf-92e2-ad7553a1a5c5

image

@AndiAJ AndiAJ closed this as completed Jul 30, 2020
@cbonacuse
Copy link
Author

@AndiAJ - it looks like the Beta screenshots are for the Search Widget event - did you try with this E_Changed_Default_To_Fenix one?

@jhugman - I tested this on my Google Pixel 3 with Nightly Build 200730 and am not seeing the event fire, even though the default_browser attribute is correctly changing from org.mozilla.firefox to org.mozilla.fenix. I tried both cold starting and also backgrounding the app, opening the app, and then killing.

@AndiAJ
Copy link
Collaborator

AndiAJ commented Jul 31, 2020

Hi @cbonacuse , my bad, mixed up the screenshots, sorry for the inconvenience. ☺️
Re-checked it today and the E_Changed_Default_To_Fenix is properly generated.

✔️ Beta

Google Pixel 3a - device ID 3d68a448-f183-4f0e-bc6e-6a3049c85d6d
image

Huawei Mate 20 Lite - device ID 2d0fe138-a652-4d72-81e3-62a60b1a7017
image

OnePlus A3 - device ID a3f8e20f-cefa-46bd-967a-c5dd963e367b
image

@sblatz
Copy link
Contributor

sblatz commented Jul 31, 2020

@cbonacuse are you still not seeing it on your end? Want to make sure this is properly resolved 😄

@cbonacuse
Copy link
Author

@sblatz thanks for following up! I am not still :(

@sblatz
Copy link
Contributor

sblatz commented Jul 31, 2020

@AndiAJ are you able to tell me the steps you used to verify this so @cbonacuse can confirm on her end?

Also are you verifying using Glean or Amplitude 🤔

@AndiAJ
Copy link
Collaborator

AndiAJ commented Aug 4, 2020

Hi @sblatz & @cbonacuse , I'm using only Leanplum.

  1. Identify the devices I use and register them as test devices.
  2. Open each test device in a separate tab.
  3. After setting for E.g. Nightly as the default browser I open a page (or perform any other action) and put Fenix in background.
  4. Refresh the tabs from Step 2
  5. Check the Timeline section

► Re-checked on today's Nightly Build

Google Pixel 3a - device ID f5921b8d-314f-40f4-aa19-97360d5d94b8
image

Huawei Mate 20 Lite - device ID af0c049d-bed4-4c99-a917-e03e7fd42d2a
image

OnePlus A3 - device ID 74f1b775-d6e0-40e9-a804-0d374398b7a5
image

@cbonacuse
Copy link
Author

@AndiAJ & @sblatz I just did another test myself with the following steps and I'm still not seeing this on my Google Pixel 3 (OS 10 & Nightly build 200805) when I do a cold start or background the app. I'm checking in Leanplum as well.

My Leanplum ID: 4ead7db7-0a20-4f5e-911b-4097b3db4048

When you say 'open device in a separate tab', are you not using an actual device but rather simulating it somehow?

  1. Check device settings - Chrome is default browser
  2. Check attribute - default_browser = null
  3. Go to device settings, set Nightly as default browser
  4. Open Nightly
  5. Check attribute - default_browser = org.mozilla.fenix
  6. Check events - no event fired except E_Opened_App
  7. Kill app
  8. Check events - no event fired
  9. Open app again
  10. Background app
  11. Open app from background
  12. Kill app

@AndiAJ AndiAJ added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Aug 6, 2020
@AndiAJ
Copy link
Collaborator

AndiAJ commented Aug 6, 2020

Hi @cbonacuse & @sblatz I've just re-checked this matter on Nightly 200805 with a Google Pixel 3a (Android 10) using the above mentioned STR [added a couple of notes and an extra step (Step 6)] and everything seems to be working fine.

Device ID - d853ec8d-3230-45a9-be08-0e147b033257

  1. Check device settings - Chrome is default browser
  2. Check attribute - default_browser = null
  3. Go to device settings, set Nightly as default browser
  4. Open Nightly
  5. Check attribute - default_browser = null (❗ it will be updated only after close + resume)
  6. Force close and resume
  7. Re-check attribute - ✔️ default_browser = org.mozilla.fenix
  8. Check events - ✔️ E_Changed_Default_To_Fenix event fired
    image
    image

@cbonacuse
Copy link
Author

@AndiAJ I followed your same steps above and I'm still not seeing the event for 4ead7db7-0a20-4f5e-911b-4097b3db4048. Could it have something to do with my device (Google Pixel 3 - yours is a 3a)?

@AndiAJ
Copy link
Collaborator

AndiAJ commented Aug 7, 2020

Hi, @cbonacuse , yes, Google Pixel 3a
Not sure why you aren't seeing the event fired, as you can see in the previous comments it worked properly on other devices (Huawei Mate 20 Lite (Android 9), OnePlus A3 (Android 6.0.1))

I've checked the timeline of your device and it dates back to the 3rd of April.
Perhaps a freshly installed Nightly build would do the trick? ☺️

@cbonacuse
Copy link
Author

@AndiAJ I can definitely try to install it again and see if that works. Is this in Beta yet as well? I can also test there.

@AndiAJ
Copy link
Collaborator

AndiAJ commented Aug 7, 2020

Hi @cbonacuse , yes it is available on today's Beta build 80.0.0-beta.4. (task index link if you want to install it)
Just checked it with a OnePlus A3 (Android 6.0.1) and it works.
image

@cbonacuse
Copy link
Author

@AndiAJ I deleted and then re-installed Nightly and found my new device ID. I'm still not having the event fire for me but I see the attribute is changing showing that Nightly is my default browser.

Device ID: 53d503bf-1812-46c0-ab26-fa42f7e00bb9
OS: 10
Device: Google Pixel 3
App version: 200805 build (I downloaded on 8/7 - has there not been an update since 8/5?)

Steps to reproduce:

  1. Uninstall Nightly
  2. Re-install Nightly
  3. Perform some actions to find my device in Leanplum (ex: add search widget and clear private data)
  4. Change device setting to set Nightly as default browser
  5. Open Nightly
  6. Kill Nightly
  7. Check attribute - default_browser = org.mozilla.fenix
  8. Check event - only E_Opened_App fired

Happy to hop on a call next week to discuss this further too @sblatz

@data-sync-user data-sync-user added the eng:qa:needed QA Needed label Aug 11, 2020
@data-sync-user data-sync-user changed the title Add new Leanplum E_Changed_Default_To_Fenix Event FNX3-15137 ⁃ Add new Leanplum E_Changed_Default_To_Fenix Event Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX3-15137 ⁃ Add new Leanplum E_Changed_Default_To_Fenix Event FNX-13125 ⁃ Add new Leanplum E_Changed_Default_To_Fenix Event Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX-13125 ⁃ Add new Leanplum E_Changed_Default_To_Fenix Event FNX2-12995 ⁃ Add new Leanplum E_Changed_Default_To_Fenix Event Aug 11, 2020
@AndiAJ AndiAJ removed the eng:qa:needed QA Needed label Aug 12, 2020
@AndiAJ AndiAJ closed this as completed Aug 12, 2020
@data-sync-user data-sync-user changed the title FNX2-12995 ⁃ Add new Leanplum E_Changed_Default_To_Fenix Event Add new Leanplum E_Changed_Default_To_Fenix Event May 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified needs:triage Issue needs triage P1 Current sprint
Projects
None yet
Development

No branches or pull requests

5 participants