-
Notifications
You must be signed in to change notification settings - Fork 20
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
feat: list groups by ids #191
Conversation
client/api_group.go
Outdated
Value *types.GroupMeta | ||
} | ||
|
||
func (m *GfSpListGroupsByGroupIDsResponse) UnmarshalXML(d *xml.Decoder, start xml.StartElement) error { |
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.
add comment ,
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.
fixed
client/api_group.go
Outdated
@@ -74,6 +74,8 @@ type Group interface { | |||
// ListGroupsByOwner returns a list of groups owned by the specified user, including those for which the user's expiration time has already elapsed | |||
// By default, the user is the sender. Other users can be set using the option | |||
ListGroupsByOwner(ctx context.Context, opts types.GroupsOwnerPaginationOptions) (*types.GroupsResult, error) | |||
// ListGroupsByGroupID list groups by group ids |
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.
remove this comment
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.
fixed
client/api_group.go
Outdated
|
||
type GfSpListGroupsByGroupIDsResponse map[uint64]*types.GroupMeta | ||
|
||
type GroupEntry struct { |
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.
add comment
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 need to expose these fields to the user, please start with lowercase letters.
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.
fixed
client/api_group.go
Outdated
@@ -568,3 +570,106 @@ func (c *client) ListGroupsByOwner(ctx context.Context, opts types.GroupsOwnerPa | |||
|
|||
return groups, nil | |||
} | |||
|
|||
type GfSpListGroupsByGroupIDsResponse map[uint64]*types.GroupMeta |
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 need to expose these fields to the user, please start with lowercase letters.
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.
fixed
Description
support list groups by ids
update payment account info in list user payment account
Rationale
N/A
Example
N/A
Changes
Notable changes: