search

botman / driver-facebook

BotMan Facebook Messenger Driver
MIT License
71 stars 72 forks source link

Added support for share_contents #35

Closed johntiror closed 6 years ago

johntiror commented 6 years ago

Here the docs from FB: https://developers.facebook.com/docs/messenger-platform/reference/buttons/share There is an issue (maybe a Facebook bug) that cause an error from FB API in case buttons have the field webview_share_button that's the reason why I had to add the function removeWebviewShareButton in order to remove that field only in case of share_contents

codecov[bot] commented 6 years ago

Codecov Report

Merging #35 into master will decrease coverage by 1.37%. The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #35      +/-   ##
============================================
- Coverage     72.45%   71.08%   -1.38%     
- Complexity      300      306       +6     
============================================
  Files            29       29              
  Lines           777      792      +15     
============================================
  Hits            563      563              
- Misses          214      229      +15
Impacted Files Coverage Δ Complexity Δ
src/Extensions/ElementButton.php 70.68% <0%> (-24.66%) 25 <6> (+6)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update d34871f...3c867a9. Read the comment docs.