-
Notifications
You must be signed in to change notification settings - Fork 230
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
bugfix: message.Type() is a empty value #333
Conversation
Signed-off-by: 1046102779 <seachen@tencent.com>
Signed-off-by: 1046102779 <seachen@tencent.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #333 +/- ##
===========================================
+ Coverage 55.79% 78.79% +23.00%
===========================================
Files 86 39 -47
Lines 5420 3565 -1855
===========================================
- Hits 3024 2809 -215
+ Misses 2174 576 -1598
+ Partials 222 180 -42 ☔ View full report in Codecov by Sentry. |
Signed-off-by: 1046102779 <seachen@tencent.com>
cc @kkdai. Help me review this PR. Thanks. |
cc @kkdai |
Hi @1046102779 |
Any update? My temporary solution: _, ok := event.Message.(*linebot.TextMessage) |
@yujinlin0224
line-bot-sdk-go/examples/kitchensink/server.go Lines 116 to 129 in 62e2928
|
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.
LGTM
issue: #332
cc @kkdai @clsung
Signed-off-by: 1046102779 seachen@tencent.com