jamesmarcogliese / slack-api-csharp Goto Github PK

When calling a List function, the class names are as follows:
ListOKResponseModel
ListOKResponseModelModel
ListOKResponseModelModelModel...
It carries on until there are nine "Model"s at the end.

Obviously not critical, but it would be better to have actual names corresponding to the action that returns them rather than something so arbitrary.

Block Kit is now the preferred method of building interactive/rich messages in Slack. It looks like the Slack API schema has been updated to include this - is it possible to regenerate the classes for this solution so that these latest updates get applied?

It looks like the only public property for RepliesOKResponse is Ok -- for example, returned when calling

var resp = await slackWebAPI.Channels.RepliesAsync(token: token, threadTs: ts, channel: chan)

Unless I'm mistaken, according to the docs here, shouldn't there also be properties related to messages and possibly has_more?

I've verified this using the test for reactions.get:

1. Using a timestamp that ends with trailing zeroes (e.g. 1549470028.002800) works fine and gets related reactions.
2. Removing the trailing zeroes (e.g. 1549470028.0028) results in the following response:

{
    "ok": false,
    "error": "message_not_found"
}

When passing the timestamp parameter as a double to a method in slack-api-csharp, for instance:

double ts = 1549470028.002800;
var resp = await slackWebAPI.Reactions.GetWithHttpMessagesAsync(token: token, channel: "the-channel", timestamp: ts);

you can see when debugging that the trailing zeroes in ts are truncated and the method returns the same message_not_found response as above.