Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add include_num_members support to conversation.info request #989

Merged
merged 7 commits into from Dec 16, 2022
Merged

Add include_num_members support to conversation.info request #989

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
2bb1377
Add include_num_members support to conversation.info request
johanmcgwire-cb Oct 21, 2021
094f096
Fix test
johanmcgwire-cb Oct 21, 2021
e4955fd
Swap to struct input
Yohan460 Nov 24, 2021
fd9eee8
gofmt
Yohan460 Feb 13, 2022
439edc3
Return errors upon missing parameters
Yohan460 May 2, 2022
6ee76f4
Use built-in errors package
Yohan460 May 3, 2022
ba92d34
combine tests to handle http handler registration
Yohan460 May 8, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
24 changes: 18 additions & 6 deletions conversation.go
View file
Open in desktop
Expand Up @@ -357,17 +357,29 @@ func (api *Client) CreateConversationContext(ctx context.Context, channelName st
return &response.Channel, nil
}

// GetConversationInfoInput Defines the parameters of a GetConversationInfo and GetConversationInfoContext function
type GetConversationInfoInput struct {
ChannelID string
IncludeLocale bool
IncludeNumMembers bool
}

// GetConversationInfo retrieves information about a conversation
func (api *Client) GetConversationInfo(channelID string, includeLocale bool) (*Channel, error) {
return api.GetConversationInfoContext(context.Background(), channelID, includeLocale)
func (api *Client) GetConversationInfo(input *GetConversationInfoInput) (*Channel, error) {
return api.GetConversationInfoContext(context.Background(), input)
}

// GetConversationInfoContext retrieves information about a conversation with a custom context
func (api *Client) GetConversationInfoContext(ctx context.Context, channelID string, includeLocale bool) (*Channel, error) {
func (api *Client) GetConversationInfoContext(ctx context.Context, input *GetConversationInfoInput) (*Channel, error) {
if input == nil {
input = &GetConversationInfoInput{}
}

Yohan460 marked this conversation as resolved.
Show resolved Hide resolved
values := url.Values{
"token": {api.token},
"channel": {channelID},
"include_locale": {strconv.FormatBool(includeLocale)},
"token": {api.token},
"channel": {input.ChannelID},
"include_locale": {strconv.FormatBool(input.IncludeLocale)},
"include_num_members": {strconv.FormatBool(input.IncludeNumMembers)},
}
response, err := api.channelRequest(ctx, "conversations.info", values)
if err != nil {
Expand Down
4 changes: 3 additions & 1 deletion conversation_test.go
View file
Open in desktop
Expand Up @@ -400,7 +400,9 @@ func TestGetConversationInfo(t *testing.T) {
http.HandleFunc("/conversations.info", okChannelJsonHandler)
once.Do(startServer)
api := New("testing-token", OptionAPIURL("http://"+serverAddr+"/"))
channel, err := api.GetConversationInfo("CXXXXXXXX", false)
channel, err := api.GetConversationInfo(&GetConversationInfoInput{
ChannelID: "CXXXXXXXX",
})
if err != nil {
t.Errorf("Unexpected error: %s", err)
return
Expand Down