Hide attendance status tag if Unknown

Author: AdamGrzybkowski

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/bookings/BookingMapper.kt

@@ -121,11 +121,11 @@ class BookingMapper @Inject constructor(
         }
     }
 
-    private fun BookingEntity.AttendanceStatus.toUiModel(): BookingAttendanceStatus = when (this) {
+    private fun BookingEntity.AttendanceStatus.toUiModel(): BookingAttendanceStatus? = when (this) {
         BookingEntity.AttendanceStatus.Booked -> BookingAttendanceStatus.Booked
         BookingEntity.AttendanceStatus.CheckedIn -> BookingAttendanceStatus.CheckedIn
         BookingEntity.AttendanceStatus.NoShow -> BookingAttendanceStatus.NoShow
-        is BookingEntity.AttendanceStatus.Unknown -> BookingAttendanceStatus.Unknown(this.key)
+        is BookingEntity.AttendanceStatus.Unknown -> null
     }
 
     private fun BookingCustomerInfo.fullName(): String? {

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/bookings/compose/BookingAttendanceSection.kt

@@ -35,7 +35,7 @@ import com.woocommerce.android.ui.compose.theme.WooThemeWithBackground
 
 @Composable
 fun BookingAttendanceSection(
-    status: BookingAttendanceStatus,
+    status: BookingAttendanceStatus?,
     attendanceUpdateStatus: AttendanceUpdateStatus,
     onClick: () -> Unit,
     modifier: Modifier = Modifier,

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/bookings/compose/BookingAttendanceStatusTag.kt

@@ -73,17 +73,16 @@ sealed interface BookingAttendanceStatus {
     data object CheckedIn : BookingAttendanceStatus
     data object NoShow : BookingAttendanceStatus
     data object Cancelled : BookingAttendanceStatus
-    data class Unknown(val key: String) : BookingAttendanceStatus
 }
 
 @Composable
-fun BookingAttendanceStatus.text(): String {
+fun BookingAttendanceStatus?.text(): String {
     return when (this) {
         BookingAttendanceStatus.Booked -> stringResource(R.string.booking_attendance_status_booked)
         BookingAttendanceStatus.CheckedIn -> stringResource(R.string.booking_attendance_status_checked_in)
         BookingAttendanceStatus.Cancelled -> stringResource(R.string.booking_attendance_status_cancelled)
         BookingAttendanceStatus.NoShow -> stringResource(R.string.booking_attendance_status_no_show)
-        is BookingAttendanceStatus.Unknown -> key
+        else -> ""
     }
 }
 

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/bookings/compose/BookingSummary.kt

@@ -57,10 +57,12 @@ fun BookingSummary(
             modifier = Modifier
                 .padding(top = 8.dp)
         ) {
-            BookingAttendanceStatusTag(
-                state = model.attendanceStatus,
-                attendanceUpdateStatus = model.attendanceUpdateStatus,
-            )
+            model.attendanceStatus?.let {
+                BookingAttendanceStatusTag(
+                    state = it,
+                    attendanceUpdateStatus = model.attendanceUpdateStatus,
+                )
+            }
             BookingStatusTag(
                 state = model.status
             )
@@ -72,7 +74,7 @@ data class BookingSummaryModel(
     val date: String,
     val name: String,
     val customerName: String?,
-    val attendanceStatus: BookingAttendanceStatus,
+    val attendanceStatus: BookingAttendanceStatus?,
     val status: BookingStatus,
     val attendanceUpdateStatus: AttendanceUpdateStatus,
 )

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/bookings/details/BookingDetailsViewModel.kt

@@ -177,7 +177,6 @@ class BookingDetailsViewModel @Inject constructor(
         BookingAttendanceStatus.Booked -> BookingEntity.AttendanceStatus.Booked
         BookingAttendanceStatus.CheckedIn -> BookingEntity.AttendanceStatus.CheckedIn
         BookingAttendanceStatus.NoShow -> BookingEntity.AttendanceStatus.NoShow
-        is BookingAttendanceStatus.Unknown,
         BookingAttendanceStatus.Cancelled -> null
     }