Skip to content

Commit 98f71c8

Browse files
committed
Merge branch 'release/7.8' into issue/4960-plugins
2 parents 653c56a + 20febbb commit 98f71c8

37 files changed

+922
-522
lines changed

WooCommerce/Classes/Tools/Logging/WooCrashLoggingStack.swift

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -116,6 +116,10 @@ class WCCrashLoggingDataProvider: CrashLoggingDataProvider {
116116
return BuildConfiguration.current.rawValue
117117
}
118118

119+
var shouldEnableAutomaticSessionTracking: Bool {
120+
return CrashLoggingSettings.didOptIn
121+
}
122+
119123
@objc func updateCrashLoggingSystem(_ notification: Notification) {
120124
/// Bumping this call to a later run loop is a little bit hack-y, but because the `StoresManager` fires the events
121125
/// we're interested as part of its initialization, we need to wait for that initalization to be complete before

WooCommerce/Classes/ViewRelated/Dashboard/Settings/SettingsViewController.swift

Lines changed: 5 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -81,10 +81,6 @@ final class SettingsViewController: UIViewController {
8181
///
8282
private var storePickerCoordinator: StorePickerCoordinator?
8383

84-
/// Flag indicating whether the currently selected store is eligible
85-
/// for card present payments
86-
private var canCollectPayments: Bool = false
87-
8884
/// Announcement for the current app version
8985
///
9086
private var announcement: Announcement?
@@ -204,21 +200,6 @@ private extension SettingsViewController {
204200
sites = sitesResultsController.fetchedObjects
205201
}
206202

207-
/// Determine if any payment gateway account for this site supports card present payments
208-
///
209-
private func checkAvailabilityForPayments() {
210-
paymentGatewayAccounts = paymentGatewayAccountsResultsController?.fetchedObjects ?? []
211-
canCollectPayments = paymentGatewayAccounts.contains(where: \.isCardPresentEligible)
212-
}
213-
214-
private var storeSettingsRows: [Row] {
215-
var result = [Row]()
216-
if canCollectPayments {
217-
result.append(.inPersonPayments)
218-
}
219-
return result
220-
}
221-
222203
func configureTableViewFooter() {
223204
// `tableView.tableFooterView` can't handle a footerView that uses autolayout only.
224205
// Hence the container view with a defined frame.
@@ -239,7 +220,6 @@ private extension SettingsViewController {
239220

240221
func refreshViewContent() {
241222
updateSites()
242-
checkAvailabilityForPayments()
243223
configureSections()
244224
tableView.reloadData()
245225
}
@@ -299,14 +279,12 @@ private extension SettingsViewController {
299279
sections.append(Section(title: pluginsTitle, rows: [.plugins], footerHeight: UITableView.automaticDimension))
300280
}
301281

302-
// Store Settings
303-
if storeSettingsRows.isNotEmpty {
304-
sections.append(
305-
Section(title: storeSettingsTitle,
306-
rows: storeSettingsRows,
307-
footerHeight: UITableView.automaticDimension)
282+
sections.append(
283+
Section(title: storeSettingsTitle,
284+
rows: [.inPersonPayments],
285+
footerHeight: UITableView.automaticDimension
308286
)
309-
}
287+
)
310288

311289
// Help & Feedback
312290
if couldShowBetaFeaturesRow() {

WooCommerce/Resources/ar.lproj/Localizable.strings

Lines changed: 57 additions & 23 deletions
Large diffs are not rendered by default.

WooCommerce/Resources/de.lproj/Localizable.strings

Lines changed: 57 additions & 23 deletions
Large diffs are not rendered by default.

WooCommerce/Resources/es.lproj/Localizable.strings

Lines changed: 57 additions & 23 deletions
Large diffs are not rendered by default.

WooCommerce/Resources/fr.lproj/Localizable.strings

Lines changed: 57 additions & 23 deletions
Large diffs are not rendered by default.

WooCommerce/Resources/he.lproj/Localizable.strings

Lines changed: 57 additions & 23 deletions
Large diffs are not rendered by default.

WooCommerce/Resources/id.lproj/Localizable.strings

Lines changed: 57 additions & 23 deletions
Large diffs are not rendered by default.

WooCommerce/Resources/it.lproj/Localizable.strings

Lines changed: 57 additions & 23 deletions
Large diffs are not rendered by default.

WooCommerce/Resources/ja.lproj/Localizable.strings

Lines changed: 57 additions & 23 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)