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

added device list that support single and double ESIM. #382

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

furiosFast
Copy link
Contributor

No description provided.

Copy link
Member

@Zandor300 Zandor300 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it possible for you to update your PR to include the dual eSim devices in the list for the single eSim devices. Could you also add tests for your changes and update from master?


/// All devices that support dual ESIM.
public static var allDevicesWithDualEsimSupport: [Device] {
return [.iPhone13, .iPhone13Mini, .iPhone13Pro, .iPhone13ProMax, .iPhoneSE3, .iPhone14, .iPhone14Plus, .iPhone14Pro, .iPhone14ProMax, .iPhone15, .iPhone15Plus, .iPhone15Pro, .iPhone15ProMax]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for my late reply. Why are the devices with dual eSim support not included in the single eSim list?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's an 11" iPad that is missing from the single sim list. Also, new 2024 iPads aren't in the list.

* master:
  Version bump. (v5.4.0)
  Add unknown orientation variant (devicekit#411)
  Version bump. (v5.3.1)
  Fix path for PrivacyInfo.xcprivacy in test target.
  Process Privacy manifest instead of copy on SPM. (devicekit#409)
  Version bump. (v5.3.0)
  Support for new May 2024 devices. (devicekit#412)
  Fix ruby version not found. (devicekit#414)
  Version bump. (v5.2.4)
  Enable MERGEABLE_LIBRARY. (devicekit#407)
  Update cocoapods to v1.15.2 in Gemfile.
  Release 5.2.3
  fix: Use cocoapod resource_bundles for PrivacyInfo (devicekit#397)
  Fix building for visionOS.  (devicekit#390)
  Fix building SwiftUI previews on macOS. (devicekit#389)
  Release 5.2.1 (devicekit#385)
  Fix PrivacyInfo.xcprivacy not being detected on SPM. (devicekit#384)
  Update swift-tools-version in Package.swift to 5.3 to support Resource files. (devicekit#381)

# Conflicts:
#	Source/Device.swift.gyb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants