Skip to content

Conversation

fabOnReact
Copy link
Owner

Summary

  • add data sync and connected device foreground service permissions
  • tighten WearConnectivityTask service attributes for FGS

Testing

  • yarn lint (fails: ESLint couldn't find the plugin "eslint-plugin-ft-flow"; installation attempt failed due to network restrictions)
  • yarn typecheck
  • yarn test (fails: Cannot find module '@react-native/babel-preset'; installation attempt failed due to network restrictions)

https://chatgpt.com/codex/tasks/task_e_68b702ef4f748320a918e27ad55fc3b8

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codex Review: Here are some suggestions.

Reply with @codex fix comments to fix any unresolved comments.

About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".

Comment on lines +6 to +10
<uses-permission android:name="android.permission.FOREGROUND_SERVICE_DATA_SYNC" />
<uses-permission android:name="android.permission.FOREGROUND_SERVICE_CONNECTED_DEVICE" />
<service android:name="com.wearconnectivity.WearConnectivityTask"
android:permission="android.permission.BIND_JOB_SERVICE"
android:exported="true" />
<intent-filter>
<action android:name="com.facebook.react.HeadlessJsTaskService" />
</intent-filter>
android:permission="android.permission.FOREGROUND_SERVICE_CONNECTED_DEVICE"
android:exported="false"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[P0] Declare connectedDevice FGS type without raising compile SDK

The service now declares android:foregroundServiceType="dataSync|connectedDevice" and requests android.permission.FOREGROUND_SERVICE_CONNECTED_DEVICE. However the module still compiles with SDK 31 (see WearConnectivity_compileSdkVersion=31). API 31 does not define the connectedDevice foreground-service enum or the new permission, so ./gradlew :android:assemble will fail with an unknown flag during manifest processing. Either remove the Android 14-only values or bump the library’s compile/target SDKs so the build system knows about these new FGS types.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant