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

mail metrics erroneously reports false #1134

Closed
incanus opened this issue Mar 27, 2015 · 2 comments
Closed

mail metrics erroneously reports false #1134

incanus opened this issue Mar 27, 2015 · 2 comments
Labels
bug iOS Mapbox Maps SDK for iOS telemetry Integration with Mapbox Telemetry libraries

Comments

@incanus
Copy link
Contributor

incanus commented Mar 27, 2015

If MFMailComposeViewController is not found, mail setup status can't be checked. It might not be found because its framework isn't linked in, but email could still be setup. The canSendMail method is only meant to check that mail is setup before attempting to send.

/cc @bleege
@incanus incanus added bug iOS Mapbox Maps SDK for iOS telemetry Integration with Mapbox Telemetry libraries labels Mar 27, 2015
@bleege
Copy link
Contributor

bleege commented Mar 28, 2015

Discussed with @camilleanne @edenh @incanus this afternoon and decided to pull the enabled.mail metric for now as the current setup "only tells us if the app is using email sending at all, not if email is setup on the device".

bleege added a commit that referenced this issue Mar 28, 2015
@bleege
#1134 - Removing enabled.email metric and supporting infrastructure
a325cb9
@bleege bleege mentioned this issue Mar 28, 2015
Merged
@bleege
Copy link
Contributor

bleege commented Mar 28, 2015

Removed and merged!

@bleege bleege closed this as completed Mar 28, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug iOS Mapbox Maps SDK for iOS telemetry Integration with Mapbox Telemetry libraries
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@incanus @bleege