-
Notifications
You must be signed in to change notification settings - Fork 71
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
Support sending and receiving stickers for WAC and D3C #610
Conversation
7e20fe4
to
1847a76
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #610 +/- ##
==========================================
+ Coverage 74.57% 74.61% +0.03%
==========================================
Files 110 110
Lines 13280 13290 +10
==========================================
+ Hits 9904 9916 +12
+ Misses 2664 2662 -2
Partials 712 712 ☔ View full report in Codecov by Sentry. |
@norkans7 since this isn't a bug gonna hold off merging until after release |
handlers/dialog360/dialog360.go
Outdated
@@ -621,7 +632,9 @@ func (h *handler) Send(ctx context.Context, msg courier.Msg, clog *courier.Chann | |||
|
|||
} else if i < len(msg.Attachments()) && (len(qrs) == 0 || len(qrs) > 3) { | |||
attType, attURL := handlers.SplitAttachment(msg.Attachments()[i]) | |||
attType = strings.Split(attType, "/")[0] | |||
splitedAttType := strings.Split(attType, "/") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@norkans7 how about we rework this to call resolve media and then we can also check that the image is 512 x 512 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, I'll update that
1847a76
to
fd6068d
Compare
contains #624 |
fd6068d
to
ef2015f
Compare
ef2015f
to
dc47057
Compare
… for media support config
This branch is out of date I will rework on that again |
dc47057
to
ba7fa36
Compare
This adds support to send and receive stickers as documented on https://developers.facebook.com/docs/whatsapp/cloud-api/reference/media#supported-media-types
The file need to be for format
image/webp
and size less than 100KB or 500KB for animated one with dimensions not exceeding 512x512