Naoki Kanatani comments

Results 59 comments of


                                            Naoki Kanatani

initial documentation for events example

@mhemken-nyt Hi, I fixed this PR in documentation/events-example branch. Could you add these fixes in this PR? git diff ```diff diff --git a/examples/eventsapi/README.md b/examples/eventsapi/README.md index 9b4e27d..af8b701 100644 --- a/examples/eventsapi/README.md +++...

Pretty-Print JSON message

Thanks for reporting! I think whether we should use pretty print or not is one thing, whether we should use application/json or not is another. What problem do you resolve?...

Pretty-Print JSON message

I'm sorry. I'm not sure about your use case. Which JSON do you want to have prettty-printed output?(Is it the body of an HTTP request sent to Slack?)

Pretty-Print JSON message

> Yes, it's the body (actual message displayed to the user on slack) of the HTTP request I want to pretty print. I understood! Thanks.

setting icon_url and thread_ts together and executing PostMessage disables icon_url

Thank you for reporting. I'll confirm later.

Fix the block ContextElements field marshaling

In this case, you can parse correctly if you fix your snippets like below. ```go // before L14 parsed := slack.ContextBlock{} // after L 14 var parsed *slack.ContextBlock ``` I...

Fix the block ContextElements field marshaling

If we change the receiver type to a value instead of a pointer, I believe this change may be applied to all `XxxBlock` types. This is to avoid misuse.

Return ScheduledMessageID everywhere

@zchee What do you think?

reactions.list not works on page-based pagination

Thank you for your reporting! Let me give some consideration to how to fix.

reactions.list not works on page-based pagination

Thanks! According to Slack support, this behaviour seems to be a bug in Slack API. So I'd like to keep page-based pagination, but make the change to support cursor-based pagination...