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

Bulk SMS BD api mobile number and message formate not correct. #6

Closed
Islamshawon71 opened this issue Feb 21, 2022 · 6 comments
Closed
Labels
bug Something isn't working issue

Comments

@Islamshawon71
Copy link

Update this on

use Xenon\LaravelBDSms\Provider\BulkSmsBD;

'username' => $config['username'],
'password' => $config['password'],
'number' => $number,
'message' => $text,

@arif98741
Copy link
Owner

arif98741 commented Feb 21, 2022

Thanks for using Xenon/LaravelBDSms package

can u share exact response please?
which version of laravelbdsms package you are using ?

@arif98741
Copy link
Owner

arif98741 commented Feb 21, 2022

According to my findings and observation I think that you are using xenon/laravelbdsms which is lower version than v1.0.38.*
From version v1.0.38.* mobile number format and length validation has been removed because sometimes it creates issue for several sms gateways.

I suggest you to upgrade your xenon/laravelbdsms to v1.0.38.* .
Use below command for upgrading your package

composer update xenon/laravelbdsms

Here is a screenshot of composer.json file that is using xenon/laravelbdsms package
32D
34ff

@Islamshawon71
Copy link
Author

I got problem with this line : Phone number

'contacts' => $number,

$response = $client->request('GET', '', [
'query' => [
'username' => $config['username'],
'password' => $config['password'],
'contacts' => $number,
'msg' => $text,
]
]);

image

image

not are same as document. thats why didn't get response properly.

@Islamshawon71
Copy link
Author

Screenshot 2022-02-22 at 12 40 32 AM

@arif98741
Copy link
Owner

Thanks for your great contribution to find out this bug. I am updating this query param by number soon and pushing a new release. Please update your package.

@arif98741 arif98741 added the bug Something isn't working label Feb 21, 2022
@arif98741
Copy link
Owner

New version v1.0.38.2 is released after fixing that bug.

@Islamshawon71
Special thanks for your nice contribution. Hope you will continue to participate in open source world in future also.
I am closing this issue soon. If you need to mention something new you can reopen it.

@arif98741 arif98741 pinned this issue Feb 21, 2022
@arif98741 arif98741 unpinned this issue Mar 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working issue
Projects
None yet
Development

No branches or pull requests

2 participants