-
Notifications
You must be signed in to change notification settings - Fork 44
Conversation
Methods for archive channel, join channel, create channel, set channel purpose and set channel topic.
Create, join, archive, set purpose and set topic for channels
@Workshop2 ready for your review. |
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.
Wow, amazing work. A couple of bits of feedback.
/// Opens a DM channel to a user. Required to PM someone. | ||
/// </summary> | ||
Task<SlackChatHub> JoinDirectMessageChannel(string user); | ||
|
||
/// <summary> | ||
/// Creates a channel. |
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.
Wrong XML comment
Task<SlackChatHub> JoinChannel(string channelName); | ||
|
||
/// <summary> | ||
/// Joins a channel. |
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.
And again
|
||
// when | ||
var result = true; | ||
try |
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.
Should let exceptions bubble up and be caught by the test harness
@Workshop2 corrections made. |
Thanks @AshleyPoole Fantastic work |
Are you able to publish the nuget package for the latest version @Workshop2? I've checked nuget just but it's showing the old previous version. |
Done |
This PR adds the ability to:
All new methods have accompanying tests using the format setup already in this repo.
This PR implements the feature request in issue (feature request) #11.