New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix issue with receivers being in the wrong location and enable multidex #1
Conversation
@@ -39,6 +39,7 @@ android { | |||
versionCode flutterVersionCode.toInteger() | |||
versionName flutterVersionName | |||
testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner" | |||
multiDexEnabled true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
multidex needs to be enabled as per https://developer.android.com/studio/build/multidex
@@ -39,5 +34,11 @@ | |||
<category android:name="android.intent.category.LAUNCHER"/> | |||
</intent-filter> | |||
</activity> | |||
<receiver android:name="com.dexterous.flutterlocalnotifications.ScheduledNotificationBootReceiver"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
receivers weren't placed in the right location
@@ -3,12 +3,7 @@ | |||
|
|||
<!-- Local Notifications --> | |||
<uses-permission android:name="android.permission.RECEIVE_BOOT_COMPLETED"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just a note that there are other permissions mentioned in the readme that hadn't been added here
@@ -18,6 +18,7 @@ allprojects { | |||
subprojects { | |||
project.configurations.all { | |||
resolutionStrategy.eachDependency { details -> | |||
// not sure why the first condition has been added. AndroidX is suppose to replace the support libraries |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see question here. i suspect that you should be able to remove the first if block
created a PR to make it easier to see what changes are needed