Turn Slack map into struct about elixir-slack HOT 7 CLOSED

Was thinking about taking on a couple of these issues. For this one, do you want to turn Slack itself into a struct, or put it somewhere else to avoid cluttering the macro-ful module, like Slack.Conn?

from elixir-slack.

Also, how do you feel other structs, like Slack.Message, Slack.User. Slack.Channel, etc?

from elixir-slack.

I would move the Slack state first, but it would probably be in its own module. I was initially thinking Slack.State but could be convinced of other names.

I think once that's done it'll make sense to re-evaluate the idea of Slack.Message, user, etc.

from elixir-slack.

Both Slack.Struct and Slack.State seem a little nondescript, but I can't think of a great alternative. Slack.API is kind of inaccurate. Slack.Conn and Slack.Client work better, but kind of already occupy a space in the domain model (websocket conn + client).

from elixir-slack.

Alternatively, flip it around such that the state is a %Slack{} struct and library gets consumed with use Slack.API (or Slack.Behaviour, Slack.Client)

from elixir-slack.

Slack.State feels pretty weird because all the callbacks end up with a Slack.State and a user-defined state right next to each other in the params.

from elixir-slack.

I don't think it's too weird. 'Slack.Client' isn't bad either.

from elixir-slack.