danielberkompas / ex_twilio

Twilio API client for Elixir
MIT License
334 stars 146 forks source link

Changes to support multi-account usage #164

Closed fcevado closed 1 year ago

fcevado commented 2 years ago

This PR introduces a bunch of changes to support multi-account usage:

Reasoning for all the changes:

I've done just a small change in ExTwilio.UrlGenerator just as an example on how it can be used throughout the codebase. Most probably it's better to inject the Config.new() default config earlier as possible, to avoid it being called multiple times for a single request.

fcevado commented 2 years ago

@danielberkompas can you please take a look to make sure that I'm in the right direction, I'd like to finish adjusting the code to use the config struct.