code4romania / code4romania/mon-vot-android-kotlin

[Improvement] Use LocalBroadcastManager to show notifications

Open
#176 0 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.