Browse Source

Fix wording issue, 'screen_qr_code_login_no_camera_permission_state_description' needs the application name.

pull/2974/head
Benoit Marty 4 months ago
parent
commit
22f8bb0b10
  1. 1
      features/login/impl/src/main/kotlin/io/element/android/features/login/impl/screens/qrcode/intro/QrCodeIntroPresenter.kt
  2. 1
      features/login/impl/src/main/kotlin/io/element/android/features/login/impl/screens/qrcode/intro/QrCodeIntroState.kt
  3. 2
      features/login/impl/src/main/kotlin/io/element/android/features/login/impl/screens/qrcode/intro/QrCodeIntroStateProvider.kt
  4. 2
      features/login/impl/src/main/kotlin/io/element/android/features/login/impl/screens/qrcode/intro/QrCodeIntroView.kt
  5. 8
      features/login/impl/src/test/kotlin/io/element/android/features/login/impl/screens/qrcode/intro/QrCodeIntroPresenterTest.kt

1
features/login/impl/src/main/kotlin/io/element/android/features/login/impl/screens/qrcode/intro/QrCodeIntroPresenter.kt

@ -59,6 +59,7 @@ class QrCodeIntroPresenter @Inject constructor( @@ -59,6 +59,7 @@ class QrCodeIntroPresenter @Inject constructor(
}
return QrCodeIntroState(
appName = buildMeta.applicationName,
desktopAppName = buildMeta.desktopApplicationName,
cameraPermissionState = cameraPermissionState,
canContinue = canContinue,

1
features/login/impl/src/main/kotlin/io/element/android/features/login/impl/screens/qrcode/intro/QrCodeIntroState.kt

@ -19,6 +19,7 @@ package io.element.android.features.login.impl.screens.qrcode.intro @@ -19,6 +19,7 @@ package io.element.android.features.login.impl.screens.qrcode.intro
import io.element.android.libraries.permissions.api.PermissionsState
data class QrCodeIntroState(
val appName: String,
val desktopAppName: String,
val cameraPermissionState: PermissionsState,
val canContinue: Boolean,

2
features/login/impl/src/main/kotlin/io/element/android/features/login/impl/screens/qrcode/intro/QrCodeIntroStateProvider.kt

@ -31,6 +31,7 @@ open class QrCodeIntroStateProvider : PreviewParameterProvider<QrCodeIntroState> @@ -31,6 +31,7 @@ open class QrCodeIntroStateProvider : PreviewParameterProvider<QrCodeIntroState>
}
fun aQrCodeIntroState(
appName: String = "AppName",
desktopAppName: String = "Element",
cameraPermissionState: PermissionsState = aPermissionsState(
showDialog = false,
@ -39,6 +40,7 @@ fun aQrCodeIntroState( @@ -39,6 +40,7 @@ fun aQrCodeIntroState(
canContinue: Boolean = false,
eventSink: (QrCodeIntroEvents) -> Unit = {},
) = QrCodeIntroState(
appName = appName,
desktopAppName = desktopAppName,
cameraPermissionState = cameraPermissionState,
canContinue = canContinue,

2
features/login/impl/src/main/kotlin/io/element/android/features/login/impl/screens/qrcode/intro/QrCodeIntroView.kt

@ -66,7 +66,7 @@ fun QrCodeIntroView( @@ -66,7 +66,7 @@ fun QrCodeIntroView(
PermissionsView(
title = stringResource(R.string.screen_qr_code_login_no_camera_permission_state_title),
content = stringResource(R.string.screen_qr_code_login_no_camera_permission_state_description),
content = stringResource(R.string.screen_qr_code_login_no_camera_permission_state_description, state.appName),
icon = { Icon(imageVector = CompoundIcons.TakePhotoSolid(), contentDescription = null) },
state = state.cameraPermissionState,
)

8
features/login/impl/src/test/kotlin/io/element/android/features/login/impl/screens/qrcode/intro/QrCodeIntroPresenterTest.kt

@ -35,7 +35,8 @@ class QrCodeIntroPresenterTest { @@ -35,7 +35,8 @@ class QrCodeIntroPresenterTest {
presenter.present()
}.test {
awaitItem().run {
assertThat(desktopAppName).isEmpty()
assertThat(appName).isEqualTo("AppName")
assertThat(desktopAppName).isEqualTo("DesktopAppName")
assertThat(cameraPermissionState.permission).isEqualTo("android.permission.POST_NOTIFICATIONS")
assertThat(canContinue).isFalse()
}
@ -69,7 +70,10 @@ class QrCodeIntroPresenterTest { @@ -69,7 +70,10 @@ class QrCodeIntroPresenterTest {
}
private fun createQrCodeIntroPresenter(
buildMeta: BuildMeta = aBuildMeta(),
buildMeta: BuildMeta = aBuildMeta(
applicationName = "AppName",
desktopApplicationName = "DesktopAppName",
),
permissionsPresenterFactory: FakePermissionsPresenterFactory = FakePermissionsPresenterFactory(),
): QrCodeIntroPresenter {
return QrCodeIntroPresenter(

Loading…
Cancel
Save