Feat: Added verify method to just verify OTP for custom use cases

[iamarpitpatidar]

Why ?

• I have country_code and mobile in different columns
• Now current validate method doesn't work coz without country_code I can't use my app and with country_code it throws error that user not found.
• for these custom cases I've created a new verify method which just verify OTP and return bool.

Is it a breaking change ?

No

[mohammad-fouladgar]

If I'm not mistaken, you only need to verify(confirm) the token. If I were you, I would add a method like onlyConfirmToken() for more code readability and interactivity and use the below code:

 public function onlyConfirmToken(bool $confirm = true): static
 {
     $this->onlyConfirm = $confirm;

     return $this;
 }

//--------- usage------------
OTP()->onlyConfirmToken()->validate('mobile','token');

Anyway, Please write a test for this PR. Thanks.

[iamarpitpatidar]

Sure lemme write some tests Get Outlook for Android<https://aka.ms/AAb9ysg>

________________________________ From: M.Fouladgar ***@***.***> Sent: Saturday, September 9, 2023 2:34:52 PM To: mohammad-fouladgar/laravel-otp ***@***.***> Cc: Arpit Patidar ***@***.***>; Author ***@***.***> Subject: Re: [mohammad-fouladgar/laravel-otp] Feat: Added verify method to just verify OTP for custom use cases (PR #11) @mohammad-fouladgar commented on this pull request.

________________________________ In src/OTPBroker.php<#11 (comment)>:

return $notifiable; } + public function verify(string $mobile, string $token): bool Could you please write a test? — Reply to this email directly, view it on GitHub<#11 (review)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AKQN47ALLHXQRRL3CR3KCRLXZQWLJANCNFSM6AAAAAA4OUB2SY>. You are receiving this because you authored the thread.Message ID: ***@***.***>

[iamarpitpatidar]

Thanks for the feedback, I'll make the amend it makes more sense. Get Outlook for Android<https://aka.ms/AAb9ysg>

________________________________ From: M.Fouladgar ***@***.***> Sent: Saturday, September 9, 2023 2:34:52 PM To: mohammad-fouladgar/laravel-otp ***@***.***> Cc: Arpit Patidar ***@***.***>; Author ***@***.***> Subject: Re: [mohammad-fouladgar/laravel-otp] Feat: Added verify method to just verify OTP for custom use cases (PR #11) @mohammad-fouladgar commented on this pull request.

________________________________ In src/OTPBroker.php<#11 (comment)>:

return $notifiable; } + public function verify(string $mobile, string $token): bool Could you please write a test? — Reply to this email directly, view it on GitHub<#11 (review)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AKQN47ALLHXQRRL3CR3KCRLXZQWLJANCNFSM6AAAAAA4OUB2SY>. You are receiving this because you authored the thread.Message ID: ***@***.***>