Skip to content

Add channel_count to subteam_created / subteam_updated events#446

Merged
seratch merged 2 commits intoslackapi:masterfrom
seratch:add-types-to-subteam-events
May 6, 2020
Merged

Add channel_count to subteam_created / subteam_updated events#446
seratch merged 2 commits intoslackapi:masterfrom
seratch:add-types-to-subteam-events

Conversation

@seratch
Copy link
Copy Markdown
Contributor

@seratch seratch commented May 5, 2020

Summary

This pull request makes newly added event.channel_count property available in subteam_created / subteam_updated events. Also, this corrects the type of user_count from String to Integer (it's a breaking change but please allow me to have it for consistency in a patch release).

Requirements (place an x in each [ ])

@seratch seratch added this to the 1.0.x milestone May 5, 2020
@seratch seratch force-pushed the add-types-to-subteam-events branch from 5c0c1cd to fa9458f Compare May 5, 2020 22:25
@codecov
Copy link
Copy Markdown

codecov Bot commented May 5, 2020

Codecov Report

Merging #446 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #446   +/-   ##
=========================================
  Coverage     84.04%   84.04%           
  Complexity     2371     2371           
=========================================
  Files           250      250           
  Lines          6376     6376           
  Branches        578      578           
=========================================
  Hits           5359     5359           
  Misses          661      661           
  Partials        356      356           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2a91976...95e802c. Read the comment docs.

@seratch seratch merged commit 2ab6588 into slackapi:master May 6, 2020
@seratch seratch deleted the add-types-to-subteam-events branch May 6, 2020 23:04
seratch added a commit that referenced this pull request May 8, 2020
* [slack-api-client] #444 okhttp 4.4 -> 4.6 uprgrade (as it's highly recommended) - thanks @seratch
* [slack-api-model] #441 #442 Fix Block Elements' object equality issues - thanks @eamelink
* [slack-api-client] #445 #448 Add admin.usergroups.* API supports - thanks @seratch
* [slack-api-model] #440 Add external data source supports for dialogs - thanks @favalos
* [slack-api-model] #437 Update composition.OptionObject to have mrkdwn - thanks @seratch
* [slack-api-model] Add teams[].team_url to admin.teams.list response - thanks @seratch
* [slack-api-model] #449 Add attachement.author_id - thanks @seratch
* [slack-app-backend] #446 Add channel_count to subteam_created / subteam_updated events - thanks @seratch
* [slack-api-client etc] #444 #450 bump patch versions of AWS SDK, Micronaut, Jetty - thanks @seratch
emanguy pushed a commit to emanguy/java-slack-sdk that referenced this pull request Jun 22, 2020
* [slack-api-client] slackapi#444 okhttp 4.4 -> 4.6 uprgrade (as it's highly recommended) - thanks @seratch
* [slack-api-model] slackapi#441 slackapi#442 Fix Block Elements' object equality issues - thanks @eamelink
* [slack-api-client] slackapi#445 slackapi#448 Add admin.usergroups.* API supports - thanks @seratch
* [slack-api-model] slackapi#440 Add external data source supports for dialogs - thanks @favalos
* [slack-api-model] slackapi#437 Update composition.OptionObject to have mrkdwn - thanks @seratch
* [slack-api-model] Add teams[].team_url to admin.teams.list response - thanks @seratch
* [slack-api-model] slackapi#449 Add attachement.author_id - thanks @seratch
* [slack-app-backend] slackapi#446 Add channel_count to subteam_created / subteam_updated events - thanks @seratch
* [slack-api-client etc] slackapi#444 slackapi#450 bump patch versions of AWS SDK, Micronaut, Jetty - thanks @seratch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant