Skip to content

Commit eea28eb

Browse files
authored
Merge pull request #789 from amaurymedeiros/notification-permission-effect
Fix ForYouScreen Compose Previews
2 parents 5d423c1 + 512930c commit eea28eb

File tree

1 file changed

+4
-0
lines changed
  • feature/foryou/src/main/java/com/google/samples/apps/nowinandroid/feature/foryou

1 file changed

+4
-0
lines changed

feature/foryou/src/main/java/com/google/samples/apps/nowinandroid/feature/foryou/ForYouScreen.kt

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,7 @@ import androidx.compose.ui.graphics.toArgb
6868
import androidx.compose.ui.layout.layout
6969
import androidx.compose.ui.platform.LocalContext
7070
import androidx.compose.ui.platform.LocalDensity
71+
import androidx.compose.ui.platform.LocalInspectionMode
7172
import androidx.compose.ui.platform.testTag
7273
import androidx.compose.ui.res.painterResource
7374
import androidx.compose.ui.res.stringResource
@@ -406,6 +407,9 @@ fun TopicIcon(
406407
@Composable
407408
@OptIn(ExperimentalPermissionsApi::class)
408409
private fun NotificationPermissionEffect() {
410+
// Permission requests should only be made from an Activity Context, which is not present
411+
// in previews
412+
if (LocalInspectionMode.current) return
409413
if (VERSION.SDK_INT < VERSION_CODES.TIRAMISU) return
410414
val notificationsPermissionState = rememberPermissionState(
411415
android.Manifest.permission.POST_NOTIFICATIONS,

0 commit comments

Comments
 (0)