-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Integration for Audience Group management #250
Conversation
53c6f59
to
e60776a
Compare
Hello @be-hase , |
@o926428377 could you fix review comments? |
Sure, I thought it would be fixed in few days or a week |
528a47c
to
6c6b984
Compare
010719c
to
f7e0e68
Compare
c7bfe23
to
d04ab51
Compare
@o926428377 Sorry for the late response, please help to rebase this PR |
dbb86f6
to
ccdc2cb
Compare
|
@louis70109 |
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.
If the spec has Request
and Response
,
please add test cases to tests/
,
let's add qualities for SDK 🙂.
For more test cases you could refer https://github.com/line/line-bot-sdk-python/tree/master/tests
I appreciate your contribution!
0bee49e
to
ebefb74
Compare
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.
Thanks
#246
https://developers.line.biz/en/reference/messaging-api/#manage-audience-group