Fixed app crash when notification permission is denied, issue #201

[ShashankDeepak]

Problem / Issue No.

• BUG: App crashes or stuck at black screen if notification permission is not given #201

Describe Problem / Root Cause

• The problem was caused because though the Notification permission was requested, the case was not handled where the user denied the notification permission.

Solution proposed

The following changes are made to improve the user experience and code quality

• The app no longer requests notification permission on startup
• In the NotificationService class the functions _scheduleNotification and showNotification handle the notification call in which if the user has denied the notification permission only then the permission is asked and if allowed then the notification is sent.
• The said functions also return a boolean value to indicate if the notification was launched or not, which can be used to handle the notification state.

Additional Information

• Relevant information regarding the solution.

Screenshots

• Original Screenshot (Problem/Issue)

error.mp4

• Updated Screenshot (Fixes/Solution)

solution.mp4

[github-actions]

Thank you for submitting your pull request! We'll review it as soon as possible.

[ShashankDeepak]

I would be grateful if you could assign a level to the issue.