seated/apus

Apus is a testable and adapter based Elixir SMS library

apus's Issues

Switch to Tesla as HTTP client

In the spirit of adapter-based clients, it would be good if the http client were configurable, too.

Tesla is the best Elixir HTTP client because it's all of them and none of them.

Add typespecs

Add a plug to allow displaying a web view of sent messages

Update README for adapter specific instructions

Now that we support more than one adapter we need to update the README to detail the config values required by the selected adapter.

Sent messages viewer error

In my my_app/lib/my_app_web/router.ex:

defmodule MyApp.Router do
  use MyApp.Web,  :router

  if Mix.env == :dev do
    forward("/sent_messages", Apus.SentMessagesViewerPlug)
  end
end

After iex -S mix, it tells:

** (UndefinedFunctionError) function Apus.SentMessagesViewerPlug.init/1 is undefined (module Apus.SentMessagesViewerPlug is not available)
    Apus.SentMessagesViewerPlug.init([])

And I checked the file, no init() and call() functions, which should be required by Plug module.

Twilio adapter test timing issue

It looks as though there is timing issue in the twilio_adapter_test file for test "deliver/2 passes request_options to hackney" as I saw this occasionally failing in travis. This probably just requires a small Process.sleep to ensure some time is taken during the test.

Add README getting started

Add code docs

Add a LICENSE

I see the license is MIT: https://github.com/seated/apus/blob/master/mix.exs#L43

But it would be good to include a LICENSE file in the root of the repo

Setup dependabot

Add test coverage reporting

Setup travis config

Add support for returning the messaging service sid in the message struct

The Apus.Message struct should have an attribute for containing the messaging service sid used to send the message when this has been configured for that instance of the sms sender.

This value would also need to be shown in the sent messages web view in place of the from field when there is no from value.

Add adapter name to the log_sent function

Update the log_sent function to include the name of which adapter is being used.