Skip to content

Commit 297742d

Browse files
committed
Remove POSIneligibleReason cases that aren't in i2.
1 parent b06a0f5 commit 297742d

File tree

2 files changed

+0
-48
lines changed

2 files changed

+0
-48
lines changed

WooCommerce/Classes/POS/TabBar/POSIneligibleView.swift

Lines changed: 0 additions & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -77,10 +77,6 @@ struct POSIneligibleView: View {
7777

7878
private var suggestionText: String {
7979
switch reason {
80-
case .notTablet:
81-
return NSLocalizedString("pos.ineligible.suggestion.notTablet",
82-
value: "Please use a tablet to access POS features.",
83-
comment: "Suggestion for not tablet: use iPad")
8480
case .unsupportedIOSVersion:
8581
return NSLocalizedString("pos.ineligible.suggestion.unsupportedIOSVersion",
8682
value: "Point of Sale requires iOS 17 or later. Please update your device to iOS 17+ to use this feature.",
@@ -105,16 +101,6 @@ struct POSIneligibleView: View {
105101
return NSLocalizedString("pos.ineligible.suggestion.featureSwitchSyncFailure",
106102
value: "Try relaunching the app or check your internet connection and try again.",
107103
comment: "Suggestion for feature switch sync failure: relaunch or check connection")
108-
case let .unsupportedCountry(supportedCountries):
109-
let countryNames = supportedCountries.map { $0.readableCountry }
110-
let formattedCountryList = ListFormatter.localizedString(byJoining: countryNames)
111-
let format = NSLocalizedString(
112-
"pos.ineligible.suggestion.unsupportedCountry",
113-
value: "POS is currently only available in %1$@. Check back later for availability in your region.",
114-
comment: "Suggestion for unsupported country with list of supported countries. " +
115-
"%1$@ is a placeholder for the localized list of supported country names."
116-
)
117-
return String.localizedStringWithFormat(format, formattedCountryList)
118104
case let .unsupportedCurrency(supportedCurrencies):
119105
let currencyList = supportedCurrencies.map { $0.rawValue }
120106
let formattedCurrencyList = ListFormatter.localizedString(byJoining: currencyList)
@@ -130,10 +116,6 @@ struct POSIneligibleView: View {
130116
return NSLocalizedString("pos.ineligible.suggestion.siteSettingsNotAvailable",
131117
value: "Check your internet connection and try relaunching the app. If the issue persists, please contact support.",
132118
comment: "Suggestion for site settings unavailable: check connection or contact support")
133-
case .featureFlagDisabled:
134-
return NSLocalizedString("pos.ineligible.suggestion.featureFlagDisabled",
135-
value: "POS is currently disabled.",
136-
comment: "Suggestion for disabled feature flag: notify that POS is disabled remotely")
137119
case .selfDeallocated:
138120
return NSLocalizedString("pos.ineligible.suggestion.selfDeallocated",
139121
value: "Try relaunching the app to resolve this issue.",
@@ -176,24 +158,6 @@ private extension POSIneligibleView {
176158
}
177159
}
178160

179-
#Preview("Unsupported country") {
180-
if #available(iOS 17.0, *) {
181-
POSIneligibleView(
182-
reason: .unsupportedCountry(supportedCountries: [.US, .GB]),
183-
onRefresh: {}
184-
)
185-
}
186-
}
187-
188-
#Preview("Not a tablet") {
189-
if #available(iOS 17.0, *) {
190-
POSIneligibleView(
191-
reason: .notTablet,
192-
onRefresh: {}
193-
)
194-
}
195-
}
196-
197161
#Preview("Unsupported iOS version") {
198162
if #available(iOS 17.0, *) {
199163
POSIneligibleView(
@@ -212,15 +176,6 @@ private extension POSIneligibleView {
212176
}
213177
}
214178

215-
#Preview("Feature flag disabled") {
216-
if #available(iOS 17.0, *) {
217-
POSIneligibleView(
218-
reason: .featureFlagDisabled,
219-
onRefresh: {}
220-
)
221-
}
222-
}
223-
224179
#Preview("Feature switch disabled") {
225180
if #available(iOS 17.0, *) {
226181
POSIneligibleView(

WooCommerce/Classes/ViewRelated/Dashboard/Settings/POS/POSTabEligibilityChecker.swift

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,15 +17,12 @@ import class Yosemite.PluginsService
1717

1818
/// Represents the reasons why a site may be ineligible for POS.
1919
enum POSIneligibleReason: Equatable {
20-
case notTablet
2120
case unsupportedIOSVersion
2221
case unsupportedWooCommerceVersion(minimumVersion: String)
2322
case siteSettingsNotAvailable
2423
case wooCommercePluginNotFound
25-
case featureFlagDisabled
2624
case featureSwitchDisabled
2725
case featureSwitchSyncFailure
28-
case unsupportedCountry(supportedCountries: [CountryCode])
2926
case unsupportedCurrency(supportedCurrencies: [CurrencyCode])
3027
case selfDeallocated
3128
}

0 commit comments

Comments
 (0)