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

Allow to passing required "verify_profile_id" #82

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mul14
Copy link

@mul14 mul14 commented May 4, 2023

Based on API documentation the verify_profile_id is mandatory.

When following examples https://github.com/team-telnyx/telnyx-php/blob/master/examples/verify_sms_demo.php, I got an error message "A required parameter was missing".

This PR will allowing to pass verify_profile_id through 3rd-arguments, like so

$verify_status = \Telnyx\Verification::submit_verification($phone_number, $verification_code, [
    'verify_profile_id' => $uuid,
]);

@kevinkorb
Copy link

Since verify_profile_id is required, it would make more sense IMO for this to be the third param, and push 'options' to the 4th param. But I'm here since I'm having the same issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants