Browse Source

Display the "verification success" Snackbar only when the state is `VerificationFlowState.Finished`.

Apply the same fix for `displayLeftRoomMessage`
pull/811/head
Benoit Marty 1 year ago
parent
commit
9079825f81
  1. 13
      appnav/src/main/kotlin/io/element/android/appnav/LoggedInEventProcessor.kt

13
appnav/src/main/kotlin/io/element/android/appnav/LoggedInEventProcessor.kt

@ -24,7 +24,7 @@ import io.element.android.libraries.matrix.api.verification.VerificationFlowStat
import io.element.android.libraries.ui.strings.CommonStrings import io.element.android.libraries.ui.strings.CommonStrings
import kotlinx.coroutines.CoroutineScope import kotlinx.coroutines.CoroutineScope
import kotlinx.coroutines.Job import kotlinx.coroutines.Job
import kotlinx.coroutines.flow.drop import kotlinx.coroutines.flow.filter
import kotlinx.coroutines.flow.launchIn import kotlinx.coroutines.flow.launchIn
import kotlinx.coroutines.flow.map import kotlinx.coroutines.flow.map
import kotlinx.coroutines.flow.onEach import kotlinx.coroutines.flow.onEach
@ -47,12 +47,15 @@ class LoggedInEventProcessor @Inject constructor(
fun observeEvents(coroutineScope: CoroutineScope) { fun observeEvents(coroutineScope: CoroutineScope) {
observingJob = coroutineScope.launch { observingJob = coroutineScope.launch {
displayLeftRoomMessage.onEach { displayLeftRoomMessage
displayMessage(CommonStrings.common_current_user_left_room) .filter { it }
}.launchIn(this) .onEach {
displayMessage(CommonStrings.common_current_user_left_room)
}
.launchIn(this)
displayVerificationSuccessfulMessage displayVerificationSuccessfulMessage
.drop(1) .filter { it }
.onEach { .onEach {
displayMessage(CommonStrings.common_verification_complete) displayMessage(CommonStrings.common_verification_complete)
}.launchIn(this) }.launchIn(this)

Loading…
Cancel
Save