code4romania / code4romania/mon-vot-android-kotlin
[Improvement] Use LocalBroadcastManager to show notifications
Open
- Dominant language
- Kotlin
- Stars
- 7
- Forks
- 26
- PR merge metrics
- No merged PRs in 30d
Description
You can use `LocalBroadcastManager` to check if can show notification (the alert dialog one) instead keeping Activity instance in Application. Could lead to memory leaks.
Contributor guide
Research direction
Search the Android project for LocalBroadcastManager, the Application-held Activity instance, and the notification alert-dialog code. First trace how the Activity is retained and how notifications are requested; done means notification behavior remains intact without retaining the Activity from Application. No file or test path is named, so identify the relevant entry points before changing anything.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- android, kotlin
- Domain
- mobile
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100