Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: wrong carrier info with deprecated method #400

Merged
merged 3 commits into from
Jun 15, 2022

Conversation

liuyang1520
Copy link
Collaborator

Summary

This PR addresses the wrong carrier information issue reported in #399:

  • introduce the condition for checking iOS 12 and use the corresponding method for retrieving carrier info
  • tested in an iPhone 13 mini device (one sim only)
  • choose to get the updated carrier information always instead of monitoring the notification event

Checklist

  • Does your PR title have the correct title format?
    Yes
  • Does your PR have a breaking change?:
    No

@liuyang1520 liuyang1520 self-assigned this Jun 15, 2022
@liuyang1520 liuyang1520 merged commit 2af05da into main Jun 15, 2022
@liuyang1520 liuyang1520 deleted the fix-wrong-carrier-info branch June 15, 2022 16:41
github-actions bot pushed a commit that referenced this pull request Jun 15, 2022
## [8.10.1](v8.10.0...v8.10.1) (2022-06-15)

### Bug Fixes

* wrong carrier info with deprecated method ([#400](#400)) ([2af05da](2af05da))
@github-actions
Copy link

🎉 This PR is included in version 8.10.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

junjie added a commit to junjie/Amplitude-iOS that referenced this pull request Sep 6, 2022
…entify-object

* commit 'f07749967fb8e628f9f2c743fab2ba6600574d85':
  chore(release): 8.12.0 [skip ci]
  feat: expose more function signature for logEvent (amplitude#409)
  chore: use the reusable workflow to create jira issue (amplitude#408)
  chore(release): 8.11.1 [skip ci]
  fix: turn requiringSecureCoding on, adjust the available version (amplitude#407)
  chore(release): 8.11.0 [skip ci]
  fix: deprecation fix (amplitude#404)
  feat: add EU SSL pinning cert, extent it to support multiple domains (amplitude#403)
  chore(release): 8.10.2 [skip ci]
  Revert "Compile with code coverage enabled (amplitude#398)" (amplitude#402)
  chore(release): 8.10.1 [skip ci]
  fix: wrong carrier info with deprecated method (amplitude#400)
  Compile with code coverage enabled (amplitude#398)
  chore(release): 8.10.0 [skip ci]
  feat: Add versionId to tracking plan data (amplitude#392)
  chore(release): 8.9.0 [skip ci]
  fix: fix release action
  Update Podfile.lock
  feat: Add support on Swift Package Manager for WatchOS (amplitude#381)
  docs: add Ampli SDK reference (amplitude#385)

# Conflicts:
#	Sources/Amplitude/AMPURLSession.m
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants