-
-
Notifications
You must be signed in to change notification settings - Fork 41
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
Add group_member_post.go #18
Add group_member_post.go #18
Conversation
33dd306
to
cb31ee0
Compare
Looks like you have some minor formatting issues as well. This is almost there though, thanks for submitting! |
member_type.go
Outdated
// Member represents an group member | ||
type Member struct { | ||
ID string `json:"id"` | ||
Email string `json:"email"` | ||
} | ||
|
||
// ResopnseAddGroupMembers represents response for added member to group | ||
type ResopnseAddGroupMembers struct { | ||
// IDs has comma-delimited, like 'xxxxxxxxxx,xxxxxxxxxx' | ||
IDs string `json:"ids"` | ||
AddedAt string `json:"added_at"` | ||
} |
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.
Are either of these types used in other API calls? If so, they can remain in this separate file, but if not, let's move them into group_member_post.go
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.
No used, I'll move to group_member_post.go
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, thanks for the contribution!
Looks like your commits are not signed. Can you please sign your commits, and then I will merge? |
08410fc
to
73b88a5
Compare
OK, signed commit pushed |
Thanks for marged! |
You're welcome, thanks for the contribution! |
Hi, Thanks for this great library.
I added support for the following Group APIs:
Inaddtional, add group member exsample.