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

fix sms controller error #153

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Conversation

suhardik
Copy link

@suhardik suhardik commented Jul 1, 2022

this fix PlatformException(failed_to_fetch_sms, Flutter Telephony: Error getting SmsManager, null, null)

@ali2236
Copy link

ali2236 commented Aug 3, 2022

This PR fixes the error! please merge and update the plugin as soon as possible

Copy link

@Mrashoo24 Mrashoo24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IT WORKED

Copy link

@devmuaz devmuaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is working perfectly, Please consider merging it because we're currently doing it locally to make it work!!

@v1ncere
Copy link

v1ncere commented Aug 26, 2022

Please merge this PR, it solves the problem.

@koukibadr
Copy link

@shounak-mulay please merge this PR to fix the ^0.2.0 version of the package

@suhardik suhardik requested review from devmuaz, ali2236, v1ncere and Mrashoo24 and removed request for devmuaz, ali2236, v1ncere and Mrashoo24 September 23, 2022 14:39
Copy link

@hamsadev hamsadev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This worked for me.

@virtualars
Copy link

Could someone please explain to me in detail how to apply this solution within the app?

@gassion88
Copy link

Может ли кто-нибудь объяснить мне подробно, как применить это решение в приложении?

I need help with this too

Copy link

@Alien-nick Alien-nick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems to be the solution we need.

Copy link

@kannankvsp kannankvsp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it worked

lesterodgt

This comment was marked as duplicate.

Copy link

@lesterodgt lesterodgt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed my problem, changing in local

Copy link

@Shameness Shameness left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Handles sms manager error

@seddik seddik mentioned this pull request Sep 11, 2023
@suhardik
Copy link
Author

suhardik commented May 3, 2024

almost 2 years have passed and this pull request has not been merged. for anyone who wants to solve this problem, you can see my comment here #149 (comment)

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.