AdvancedAndroid_Shushme icon indicating copy to clipboard operation
AdvancedAndroid_Shushme copied to clipboard

Ringer mode issues

Open gaelmarhic opened this issue 7 years ago • 3 comments

Hi!

When trying this solution final code on my phone (running with Android N), I ran into some issues which made the app crash:

  1. Some packages in the GeofenceBroadcastReceiver file have not been imported, so that it does not let compile.

  2. The permission for the ACCESS NOTIFICATION POLICY is not present in the Manifest.xml file.

  3. Since when we want to change the Ringer mode we want to make sure that Notification Policy Access is granted (in case our SDK is 24 or greater), I think the appropriate code should be:

if (android.os.Build.VERSION.SDK_INT < 24 || (android.os.Build.VERSION.SDK_INT >= 24 && nm.isNotificationPolicyAccessGranted())) ...

and not if (android.os.Build.VERSION.SDK_INT < 24 || (android.os.Build.VERSION.SDK_INT >= 24 && !nm.isNotificationPolicyAccessGranted())) ...

I actually removed the "!", since in the current version of the code we try to change the Ringer mode status when the permission is not granted...

Thanks for your great tutorials guys!

Kind regards,

Gaël

gaelmarhic avatar May 21 '17 17:05 gaelmarhic

Add <uses-permission android:name="android.permission.ACCESS_NOTIFICATION_POLICY"/> in the manifest file.

dharanad avatar May 31 '17 11:05 dharanad

All bugs fixed in - https://github.com/udacity/AdvancedAndroid_Shushme/pull/12

hrishikesh-kadam avatar Dec 09 '17 14:12 hrishikesh-kadam

thanks all but i still have an issue with the ringer issue, when the geofences switch is disabled the silent mode does not get disabled and the phone still continue on silent mode? any suggestion ?

moharidy avatar Oct 08 '18 00:10 moharidy