Closed Bug 1403534 Opened 7 years ago Closed 3 years ago

Primary email not changed in settings on Android devices

Categories

(Firefox for Android Graveyard :: Firefox Accounts, defect, P5)

defect

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: sflorean, Unassigned)

Details

Attachments

(1 file)

Devices: Samsung Galaxy Note 4 (Android 5.0.1), Motorola Nexus 6 (Android 7.1.1), Oneplus Two (Android 6.0.1). 
Environment: Staging server [Train 86]
Feature: change email.

Steps to reproduce:
1. On device1, create and verify account with email X;
2. On device1, Login via sync preferences with email X;
3. On device1, Add and verify a secondary email Y;
4. On device2, Login via sync preferences with email X then close settings;
5. On device1, append `&canChangeEmail=true` to url and refresh page;
6. On device1, Click `Make Primary` on email Y";
7. Check devices sync settings.

Actual results:
On both devices - sync menu and in System Settings->Accounts the primary email was not changed. 

Expected results:
On Android devices - sync menu, we see email Y as new primary email and also in System Settings->Accounts, Firefox account was renamed.
The sync continues to function as before.
Which Fennec build were you testing with? First build with all of the follow-ups is 20170921220243.
Component: QA: General → Firefox Accounts
Flags: needinfo?(sorina.florean)
Product: Cloud Services → Firefox for Android
(In reply to :Grisha Kruglov from comment #1)
> Which Fennec build were you testing with? First build with all of the
> follow-ups is 20170921220243.

Hi, Build 20170926100255. On all Android devices.
Flags: needinfo?(sorina.florean)
Hey Sorina/Grisha,

I couldn't find any docs on how to find the exact version number but used Firefox Beta, 2017 Oct 1. I tested with Galaxy Note 4 (Android 6) and Galaxy S6 (Android 6, genymotion emulator).

From https://bugzilla.mozilla.org/show_bug.cgi?id=1403534#c0, I could not reproduce the error. When changing the primary email on either device, I could see the update email after going to settings.

However, when using the physical Android device and changing the primary email from that device, the application would crash shortly after. Upon a restarting the app, the new primary email address would be shown and everything appears to work as expected. Is there a way to get a stacktrace when using beta version, or can you only submit a crash report?
Attached file debug.txt
Attached a debug log I captured using Firefox WebIDE. It was taken right before application crashed.
Hi Vijay,

Seems like Comment 4 slipped through my bugmail. Some crashes were very recently fixed and uplifted to beta about three days ago (Bug 1404124, Bug 1403787). I'm hoping that these uplifts would fix the issues you were observing in Comment 4.

Can you still reproduce problems if you test against a nightly build?

Unfortunately the webIDE logs won't help us here. Can you get a local android dev environment up and running, and perform the same tests with 'adb logcat' running? Some "how to get the logs" instructions are found here: https://developer.mozilla.org/en-US/docs/Mozilla/QA/Fennec_Android:_Logging_with_the_Android_Debug_and_Logcat - but I reckon you'll be better off setting up a full fennec dev environment, so take the few extra steps outlined here: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Build_Instructions/Simple_Firefox_for_Android_build - specifically the artifact build ones.

I'm going to debug this a bit and see what's going on.
Flags: needinfo?(vbudhram)
The crash is likely the same as Bug 1401318.
Hey Grisha,

Thanks for the links, I was able to get an artifact build of latest Android running on my device. When using a developer build, I could change the primary email and the app would not crash.

> Can you still reproduce problems if you test against a nightly build?

I tested with today's Nightly (10/10) from Play Store and it still does crash. Note, it appears to work correctly if the change email action is performed on another device (old device shows the updated email).
Flags: needinfo?(vbudhram)
Re-triaging per https://bugzilla.mozilla.org/show_bug.cgi?id=1473195

Needinfo :susheel if you think this bug should be re-triaged.
Priority: -- → P5
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → INCOMPLETE
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: