Skip to content
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

Blocking Calls on Android #464

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

krrish-sehgal
Copy link
Contributor

@krrish-sehgal krrish-sehgal commented Nov 19, 2024

PR1 for #429

Description

Usage of method channel to invoke blocking functionality, that rejects and silences the incoming calls.

  • Method Channel Implementation and Control Flow:
    -A common method channel made names coms.apps.blt/channel and updated in report_bug as well to ensure clipboard paste still works.
    -Asking of permissions
    -Handling spam list updates
    -Registering a SpamCallBlockerService to run in the background that intercepts the calls.

    New Dependencies in gradle folder

The following dependencies are required for this change:

  • org.greenrobot:eventbus:3.2.
  • com.jakewharton.timber:timber:4.7.1
  • pub.devrel:easypermissions:3.0.0

To make sure it properly runs , we currently have to paste this in terminal manually:
" adb shell cmd role add-role-holder android.app.role.CALL_SCREENING com.apps.blt"
In time, i would like to find a programmatic approach for this.

Attached video demo of the fix

Screen.Recording.2024-11-19.at.8.28.03.PM.mov

@krrish-sehgal
Copy link
Contributor Author

@DonnieBLT , any feedbacks for me ?

@DonnieBLT
Copy link
Collaborator

Looks good! I will review further soon. Thank you.

@krrish-sehgal
Copy link
Contributor Author

krrish-sehgal commented Nov 21, 2024

@DonnieBLT sir, are there any more fixes or should i work on the failing test case, as the ios implementation for it is also ready.

@krrish-sehgal
Copy link
Contributor Author

krrish-sehgal commented Nov 26, 2024

Hi @DonnieBLT sir, just following up on this PR. Let me know if there's anything I can clarify or assist with to help with the review process. Thanks!

@krrish-sehgal krrish-sehgal mentioned this pull request Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants