feat app: Lower min SDK to API 26 #42

FlynnD273 · 2026-01-22T08:14:48Z

No description provided.

matejdro · 2026-01-22T08:20:13Z

app/build.gradle.kts

      versionName = "1.0.0"

      testInstrumentationRunner = "com.matejdro.micropebble.instrumentation.TestRunner"
+      minSdk = 26


This is not necessary, it will reuse the nubmer from the android-module-commons.gradle.kts

matejdro · 2026-01-22T08:20:16Z

apps/ui/build.gradle.kts

   buildFeatures {
      androidResources = true
   }
+   defaultConfig {


This is not necessary, it will reuse the nubmer from the android-module-commons.gradle.kts

matejdro · 2026-01-22T08:20:35Z

I think this is a good start, but I'm almost sure that it will not work as-is because app uses a lot of APIs that are not available in pre-31

FlynnD273 · 2026-01-22T08:23:33Z

I couldn't get it to crash in the emulator, but I will test further on actual hardware in the next couple of days. I'm pretty sure I have an Android 11 phone somewhere.

matejdro · 2026-01-22T09:42:04Z

(I hope you don't mind me pushing a CI fix to your branch)

EDIT: Sorry, I thought I could fix it, but there is some other underlying issue. Until I get to it, you will unfortunately have to check the action logs to see the failed lint issues.

into lower-min-sdk

… lower-min-sdk

FlynnD273 · 2026-01-23T06:59:15Z

Alright. Android 11 now works, pairs to the watch and all that. Newer Android also works too still, currently running this build on my phone.

matejdro · 2026-01-23T07:07:34Z

bluetooth/ui/src/main/kotlin/com/matejdro/micropebble/bluetooth/scan/BluetoothScanScreen.kt

               }
            }

            val bluetoothPermission = rememberMultiplePermissionsState(


Can we try moving this into a separate method? Now it's a bit long to be nested inside Composable

To be honest, I'm not sure how I'd do that easily 😅 I'll probably take another crack at it tomorrow when I've got more sleep

Looks good to me

FlynnD273 · 2026-01-23T17:59:08Z

How do I re-run the CI check? It looks like npm failed, not any linting error this time.

matejdro · 2026-01-23T20:29:20Z

How do I re-run the CI check? It looks like npm failed, not any linting error this time.

Not sure if you have permissions, but the button is here:

FlynnD273 · 2026-01-23T22:12:54Z

Should double-check the screenshot tests — I think they're correct now but I'm not entirely certain. Also the installed watchapps screen shows a bar about a third of the width of the column instead of the full bar like before, it's a side effect of switching to PrimaryTabRow instead of the deprecated TabRow

FlynnD273 · 2026-01-24T01:24:33Z

Should double-check the screenshot tests — I think they're correct now but I'm not entirely certain. Also the installed watchapps screen shows a bar about a third of the width of the column instead of the full bar like before, it's a side effect of switching to PrimaryTabRow instead of the deprecated TabRow

Actually it looks like the screenshot changes also include things gaining an accent colour and other things losing the accent colour. I'm not sure what's up with that, obviously it's a change due to this PR. Maybe the change to Theme.kt? I don't know what Android version the screenshots are generated on, could that have changed? I'm unfamiliar with how the screenshots are actually generated

matejdro · 2026-01-24T06:17:51Z

Yeah screenshots look fine. My guess is that screenshots are made on a fake lower SDK version, so the colors are slightly different.

What concerns me, are the two things using the ComponentFactory, that got disabled on the lower SDK versions without providing an alternative:

the boot receiver com.matejdro.micropebble.bluetooth.service.ServiceBootStarter - can you check that there are no errors/crashes on pre-android-9 on phone boot?
the keepalive service com.matejdro.micropebble.bluetooth.service.WatchConnectionService - can you check that the app does not crash or throw any errors when connecting to a watch on pre-android-9 device?

If you do not have pre-android-9 device to test, we should increase the min SDK back to a device that we do have and we can test it works correctly (presumably at most Android 11 device?).

FlynnD273 · 2026-01-24T17:33:47Z

Hmm I think the oldest device I have is an Android 11 phone. Maybe I bump up the minSDK to 28 instead?

matejdro · 2026-01-24T20:15:51Z

Yes, I think that would be the best.

… lower-min-sdk

github-actions · 2026-01-25T03:59:13Z

Lint report

Results

[WARNING] [ObsoleteSdkInt] Unnecessary; SDK\_INT is always >= 30
- microPebble/app/src/main/kotlin/com/matejdro/micropebble/MicroPebbleApplication.kt
  
  Line 48 in b4745ae
  
  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.P) {
- microPebble/app/src/main/kotlin/com/matejdro/micropebble/MicroPebbleApplication.kt
  
  Line 148 in b4745ae
  
  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.P) {
- microPebble/app/src/main/kotlin/com/matejdro/micropebble/MicroPebbleApplication.kt
  
  Line 169 in b4745ae
  
  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
- microPebble/app/src/main/kotlin/com/matejdro/micropebble/MicroPebbleApplication.kt
  
  Line 191 in b4745ae
  
  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.P) {
- microPebble/notification/data/src/main/kotlin/com/matejdro/micropebble/notifications/NotificationsStatusImpl.kt
  
  Line 46 in b4745ae
  
  get() = if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O_MR1) {
- microPebble/home/ui/src/main/kotlin/com/matejdro/micropebble/onboarding/OnboardingScreen.kt
  
  Line 208 in b4745ae
  
  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
- microPebble/common-compose/src/main/kotlin/com/matejdro/micropebble/ui/lists/ReorderableList.kt
  
  Line 69 in b4745ae
  
  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
- microPebble/common-compose/src/main/kotlin/com/matejdro/micropebble/ui/lists/ReorderableList.kt
  
  Line 100 in b4745ae
  
  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
- microPebble/common-compose/src/main/kotlin/com/matejdro/micropebble/ui/lists/ReorderableList.kt
  
  Line 69 in b4745ae
  
  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
- microPebble/common-compose/src/main/kotlin/com/matejdro/micropebble/ui/lists/ReorderableList.kt
  
  Line 100 in b4745ae
  
  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
- microPebble/home/ui/src/main/kotlin/com/matejdro/micropebble/onboarding/OnboardingScreen.kt
  
  Line 208 in b4745ae
  
  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
- microPebble/notification/data/src/main/kotlin/com/matejdro/micropebble/notifications/NotificationsStatusImpl.kt
  
  Line 46 in b4745ae
  
  get() = if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O_MR1) {

Suppressed Results

Nothing here.

github-actions · 2026-01-25T04:41:10Z

Lint report

Results

[WARNING] [ObsoleteSdkInt] Unnecessary; SDK\_INT is always >= 30
- microPebble/app/src/main/kotlin/com/matejdro/micropebble/MicroPebbleApplication.kt
  
  Line 161 in f528afa
  
  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
- microPebble/notification/data/src/main/kotlin/com/matejdro/micropebble/notifications/NotificationsStatusImpl.kt
  
  Line 46 in f528afa
  
  get() = if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O_MR1) {
- microPebble/home/ui/src/main/kotlin/com/matejdro/micropebble/onboarding/OnboardingScreen.kt
  
  Line 208 in f528afa
  
  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
- microPebble/common-compose/src/main/kotlin/com/matejdro/micropebble/ui/lists/ReorderableList.kt
  
  Line 69 in f528afa
  
  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
- microPebble/common-compose/src/main/kotlin/com/matejdro/micropebble/ui/lists/ReorderableList.kt
  
  Line 100 in f528afa
  
  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
- microPebble/common-compose/src/main/kotlin/com/matejdro/micropebble/ui/lists/ReorderableList.kt
  
  Line 69 in f528afa
  
  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
- microPebble/common-compose/src/main/kotlin/com/matejdro/micropebble/ui/lists/ReorderableList.kt
  
  Line 100 in f528afa
  
  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
- microPebble/home/ui/src/main/kotlin/com/matejdro/micropebble/onboarding/OnboardingScreen.kt
  
  Line 208 in f528afa
  
  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
- microPebble/notification/data/src/main/kotlin/com/matejdro/micropebble/notifications/NotificationsStatusImpl.kt
  
  Line 46 in f528afa
  
  get() = if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O_MR1) {

Suppressed Results

Nothing here.

matejdro · 2026-01-25T08:01:16Z

Thanks!

FlynnD273 mentioned this pull request Jan 22, 2026

Is Android 11 support on the roadmap? #24

Open

matejdro reviewed Jan 22, 2026

View reviewed changes

FlynnD273 force-pushed the lower-min-sdk branch from 7dafe78 to 4b31684 Compare January 22, 2026 08:22

feat app: Lower min SDK to API 26

4b31684

ci: attempt to fix bot not posting lint issues

2f47082

matejdro and others added 7 commits January 22, 2026 11:02

ci: also grant pull request write permission

2ec1b10

Get Android 11 working

38c2091

Merge branch 'lower-min-sdk' of https://github.com/FlynnD273/microPebble

ac2e665

into lower-min-sdk

ci: attempt to fix bot not posting lint issues

56c6225

ci: also grant pull request write permission

9a87a67

Merge branch 'lower-min-sdk' of https://github.com/FlynnD273/microPebble

e891faf

into lower-min-sdk

Merge branch 'master' of https://github.com/matejdro/microPebble into…

0ea9a49

… lower-min-sdk

matejdro reviewed Jan 23, 2026

View reviewed changes

Extract bluetooth scan state to function and fix one more linting issue

4b41971

FlynnD273 force-pushed the lower-min-sdk branch from ecde32f to 4b41971 Compare January 23, 2026 18:45

feat app: Lint passes now

8ae7e91

github-actions bot added 3 commits January 23, 2026 13:32

chore: add new screenshot tests

410f61c

chore: add new screenshot tests

600082c

Remove backup file

dcaabb6

github-actions bot added 2 commits January 24, 2026 19:26

Merge branch 'master' of https://github.com/matejdro/microPebble into…

d33d420

… lower-min-sdk

Raise minSdk to 30

799fe83

Remove API 26 check

2d49b5b

Actually pass lint this time

209d91a

matejdro merged commit 51bbb4c into matejdro:master Jan 25, 2026
1 check passed

feat app: Lower min SDK to API 26 #42

feat app: Lower min SDK to API 26 #42

Uh oh!

Conversation

FlynnD273 commented Jan 22, 2026

Uh oh!

matejdro Jan 22, 2026

Choose a reason for hiding this comment

Uh oh!

matejdro Jan 22, 2026

Choose a reason for hiding this comment

Uh oh!

matejdro commented Jan 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

FlynnD273 commented Jan 22, 2026

Uh oh!

matejdro commented Jan 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

FlynnD273 commented Jan 23, 2026

Uh oh!

matejdro Jan 23, 2026

Choose a reason for hiding this comment

Uh oh!

FlynnD273 Jan 23, 2026

Choose a reason for hiding this comment

Uh oh!

matejdro Jan 23, 2026

Choose a reason for hiding this comment

Uh oh!

FlynnD273 commented Jan 23, 2026

Uh oh!

matejdro commented Jan 23, 2026

Uh oh!

FlynnD273 commented Jan 23, 2026

Uh oh!

FlynnD273 commented Jan 24, 2026

Uh oh!

matejdro commented Jan 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

FlynnD273 commented Jan 24, 2026

Uh oh!

matejdro commented Jan 24, 2026

Uh oh!

github-actions bot commented Jan 25, 2026

Lint report

Results

Suppressed Results

Uh oh!

github-actions bot commented Jan 25, 2026

Lint report

Results

Suppressed Results

Uh oh!

Uh oh!

matejdro commented Jan 25, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

matejdro commented Jan 22, 2026 •

edited

Loading

matejdro commented Jan 22, 2026 •

edited

Loading

matejdro commented Jan 24, 2026 •

edited

Loading