В некоторых проектах (SaaS) может быть несколько подключений к CloudPayments#16
В некоторых проектах (SaaS) может быть несколько подключений к CloudPayments#16dapi wants to merge 7 commits intoundr:masterfrom
Conversation
| #### Local configuration: | ||
|
|
||
| ```ruby | ||
| config = CloudPayments::Config.configure do |c| |
There was a problem hiding this comment.
Я бы предложил более нативный вариант:
config = CloudPayments::Config.new do |c|
c.bla_bla_bla ...
endи CloudPayments::Config.new(hash) и CloudPayments::Config.new(file_name)
ну и комбинированный вариант
config = CloudPayments::Config.new(hash_or_string) do |c|
c.bla_bla_bla ...
endТо есть если передан hash, то используем его в качестве конфига, если передана строка, то используем ее в как имя YAML файла в котором лежит кофиг
There was a problem hiding this comment.
config = CloudPayments::Config.new
Не хотелось бы менять API уже существующего класса (который работает до нашего PR). Именно поэтому я добавил отдельный метод configure иначе этот gem потеряет обратную совместимость.
|
этот функционал нужен |
|
@unloved привет ) |
|
опечатка для вебхуков тоже надо ридми дополнить наверное |
|
Я не в состоянии больше поддерживать этот гем. Прощу прощения. |
|
Мы добавили поддержку локальной конфигурации с похожим интерфейсом в форк: https://github.com/platmart/cloud_payments#local-configuration |
Делаем возможность указать индивидуальный конфиг для Client и Webhooks