Skip to content

Commit 5af9ef9

Browse files
committed
add comments and accessibility id
1 parent af09aca commit 5af9ef9

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

WooCommerce/Classes/ViewRelated/Orders/Order Details/Order Summary Section/SummaryTableViewCell.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ final class SummaryTableViewCell: UITableViewCell {
1212
///
1313
@IBOutlet private weak var subtitleLabel: UILabel!
1414

15-
///
15+
/// Shows the sales channel if appropiate, at the moment only Point of Sale
1616
///
1717
@IBOutlet private weak var salesChannelLabel: UILabel!
1818

@@ -105,7 +105,7 @@ private extension SummaryTableViewCell {
105105
if ServiceLocator.featureFlagService.isFeatureFlagEnabled(.pointOfSaleOrdersi1) {
106106
salesChannelLabel.isHidden = false
107107
salesChannelLabel.applyFootnoteStyle()
108-
salesChannelLabel.accessibilityIdentifier = ""
108+
salesChannelLabel.accessibilityIdentifier = "summary-table-view-cell-sales-channel-label"
109109
} else {
110110
salesChannelLabel.isHidden = true
111111
}

WooCommerce/Classes/ViewRelated/Orders/Order Details/Order Summary Section/SummaryTableViewCellViewModel.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ struct SummaryTableViewCellViewModel {
4949
return formatter.string(from: dateCreated)
5050
}
5151

52-
///
52+
/// Textual representation of the sales channel
5353
///
5454
var formattedSalesChannel: String? {
5555
guard let salesChannel = salesChannel else {

0 commit comments

Comments
 (0)