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.

Sign up for GitHub

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

Jump to bottom

Verify with whatsapp should pass the $from variable through #467

Closed
stefanfr opened this issue Feb 7, 2024 · 2 comments
Closed

Verify with whatsapp should pass the $from variable through #467

stefanfr opened this issue Feb 7, 2024 · 2 comments
Assignees
@SecondeJK
Labels
enhancement work-scheduled This label indicates that the fix/resolution is scheduled into Vonage's workload

Comments

@stefanfr
Copy link

stefanfr commented Feb 7, 2024

https://github.com/Vonage/vonage-php-sdk-core/blame/8cc9df07499795400b6fc2511e77e4b0734821e5/src/Verify2/Request/WhatsAppRequest.php#L20

The $from variable should be passed through to the VerificationWorkflow class. Else this results in Invalid params: The value of one or more parameters is invalid. See https://www.nexmo.com/messages/Errors#InvalidParams for more information
@SecondeJK
Copy link
Contributor

This is a very recent change, and is in my workload to correct (from is now mandatory as you must use your own WABA to use Verify with WhatsApp)

@SecondeJK SecondeJK self-assigned this Feb 8, 2024
@SecondeJK SecondeJK added enhancement work-scheduled This label indicates that the fix/resolution is scheduled into Vonage's workload labels Feb 8, 2024
@SecondeJK
Copy link
Contributor

#469 has now been released as 4.6.3, thank you for reporting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SecondeJK SecondeJK

Labels
enhancement work-scheduled This label indicates that the fix/resolution is scheduled into Vonage's workload
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@stefanfr @SecondeJK