Closed Bug 1393492 Opened 7 years ago Closed 7 years ago

"Mailto Is Default" reads "False" on clean install and unregistered device and it should read "True"

Categories

(Firefox for iOS :: General, defect, P1)

All
iOS
defect

Tracking

()

VERIFIED FIXED
Iteration:
1.29
Tracking Status
fxios 8.3 ---
fxios-v8.3 --- verified
fxios-v10.0 --- verified

People

(Reporter: SimonB, Unassigned)

References

Details

(Whiteboard: [MMA] [MobileCore])

Attachments

(1 file)

55 bytes, text/x-github-pull-request
jhugman
: review-
farhan
: review+
Details | Review
Build: master bbaa76973a
Device: iPhone 7
iOS: 10.3.3

Steps to reproduce: 
1. Clean install Firefox on an unregistered device
2. Launch Firefox
3. Register the device to Leanplum
4. Check the "User attributes" from the leanplum dashboard 
5. Go to the iOS device and check the "Open with" settings

Actual results: 
- The "Mailto is Default" attributes will read "False" although the mailing app is the default one. After changing the default mailing app to one of choice and back to the default one, the "Mailto is default" attributes will be "True" and will change correctly.

Expected results: 
-  The "Mailto is Default" should be "True" on a clean install.
Probably needs to fix some logic somewhere.
Attached file PR
Attachment #8901992 - Flags: review?(sdaswani)
Attachment #8901992 - Flags: review?(sdaswani)
Attachment #8901992 - Flags: review?(jhugman)
Attachment #8901992 - Flags: review?(fpatel)
Comment on attachment 8901992 [details] [review]
PR

Please see :garvan's comments, and re-review.
Attachment #8901992 - Flags: review?(jhugman) → review-
Iteration: --- → 1.29
Priority: -- → P1
Whiteboard: [MMA] → [MMA] [MobileCore]
master https://github.com/mozilla-mobile/firefox-ios/commit/b259426ff1a5a5dd35eae5ee2cfe5a87feda5925
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Attachment #8901992 - Flags: review?(fpatel) → review+
uplifted to v8.x and v9.x
Verifying as fix on master b259426f.
Verifying as fix on beta 8.3(5817).
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: