Verified and tested the patch on 9.5.x. Now WhatsApp is not showing blank but on the popup, it is showing this ->
This link is incorrect. Close this window and try a different link.
Refer to the after_patch screenshot.- 🇮🇳India Raveen Kumar
@AdamPS,
I have reviewed & Implemented the Patch on my Drupal website having version 9.5, PHP version - 8.1
The patch has been implemented successfully. Can be moved to RTBC!
Please & Thank You. - Status changed to RTBC
almost 2 years ago 6:24am 16 May 2023 - 🇮🇳India Raveen Kumar
I am attaching the screenshots for reference. Please have a look (before-patch.png, after-patch.png). Moving to RTBC!
And Thank You. - last update
almost 2 years ago 1 pass - Status changed to Needs work
almost 2 years ago 9:00am 16 May 2023 - 🇬🇧United Kingdom adamps
Thanks.
- It seems that the current API is
https://wa.me/?text=urlencodedtext
. The patch seems to be using the old API format. Please see this link https://stackoverflow.com/questions/29218378/mobile-website-whatsapp-but... and other pages on the Internet. - The text should be URL-encoded, so use %20 instead of space.
- It seems that the current API is
- 🇮🇳India Raveen Kumar
@Adam, On the hover link of the icon Do you want %20 instead of % ?
- First commit to issue fork.
- last update
almost 2 years ago 1 pass - @bharath-kondeti opened merge request.
- Status changed to Needs review
almost 2 years ago 11:07am 16 May 2023 - last update
almost 2 years ago 1 pass - Status changed to Needs work
almost 2 years ago 12:50pm 16 May 2023 - 🇮🇳India Raveen Kumar
Adding test screenshot as per #10, #12. Thank You.
- Status changed to Needs review
almost 2 years ago 2:04pm 17 May 2023 - last update
almost 2 years ago 1 pass - Status changed to Fixed
almost 2 years ago 2:11pm 17 May 2023 - 🇬🇧United Kingdom adamps
Great thanks. Sure you deserve credit, now fixed - unfortunately the system doesn't seem to assign credit for reviewers automatically.
Automatically closed - issue fixed for 2 weeks with no activity.