Daniel (dB.) Doubrovkine

Results 3350 comments of Daniel (dB.) Doubrovkine

I have a whole bot that does that ;) https://github.com/dblock/slack-moji/blob/master/slack-moji/models/user.rb#L167 The value of `profile` needs to be JSON. Go figure. Would appreciate a patch ala https://github.com/slack-ruby/slack-ruby-client/blob/master/lib/slack/web/api/patches/chat.2.attachments-json.patch in this project for...

It would be nice to get a better error, although I believe that's what slack returns and we just pass it through. PRs welcome, I'll leave it open.

I support this change. It should be probably similar to [what we did in Grape](https://github.com/ruby-grape/grape/pull/1610), so maybe not OpenStruct.

I don't really have any strong feelings about `ActiveSupport::HashWithIndifferentAccess` over `OpenStruct`.

Doesn't look possible right now without patching the code. It would probably be different for web and RTM APIs. For web, we build the Faraday connection for the web client...

Aha, that's the underlying library and seems to do it. Be careful though because that disables it globally so YMMV.

I'd be down with splitting the gems and renaming the cli executable there. I don't think I want to do all the work though, so if you're into it @kwstannard,...

I'm fine with this, but i'd prefer we split the tool, too, instead of doing a backwards incompatible rename because it is the best long term solution and reduces the...

The pagination wrapper is applied [here](https://github.com/slack-ruby/slack-ruby-client/blob/6b559c34b22d18910f6ec950c6e2cc2e40239d2a/lib/slack/web/api/templates/method.erb#L51) as long as the API supports pagination, which is when it includes a `cursor` parameter. So hunt down whether that's the case, and make...

This API does not paginate like other APIs and [takes](https://api.slack.com/methods/files.list) a `page` and a `count` parameter, so it will need a different implementation.