Skip to content

Conversation

dapi
Copy link
Contributor

@dapi dapi commented Aug 28, 2017

Делаем возможность указать индивидуальный конфиг для Client и Webhooks

#### Local configuration:

```ruby
config = CloudPayments::Config.configure do |c|
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я бы предложил более нативный вариант:

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 файла в котором лежит кофиг

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

config = CloudPayments::Config.new 

Не хотелось бы менять API уже существующего класса (который работает до нашего PR). Именно поэтому я добавил отдельный метод configure иначе этот gem потеряет обратную совместимость.

@unloved
Copy link

unloved commented Dec 5, 2017

этот функционал нужен
@dapi как хорошо что ты столкнулся с этим раньше меня :D

@dapi
Copy link
Contributor Author

dapi commented Dec 6, 2017

@unloved привет )

@unloved
Copy link

unloved commented Dec 6, 2017

опечатка
client = CloudPayments::Client.new(config)

для вебхуков тоже надо ридми дополнить наверное

@undr
Copy link
Owner

undr commented Dec 25, 2017

Я не в состоянии больше поддерживать этот гем. Прощу прощения.

@kirillplatonov
Copy link

Мы добавили поддержку локальной конфигурации с похожим интерфейсом в форк: https://github.com/platmart/cloud_payments#local-configuration
Наша компания активно использует CloudPayments и мы будет поддерживать этот форк.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants