search

Velhron / dadata-bundle

Бандл для работы с API ДаДата.
MIT License
19 stars 13 forks source link

added setting dadata url in bundle configuration #5

Closed AlexandrErohin closed 3 years ago

AlexandrErohin commented 3 years ago

Если у вас инфраструктура состоит из n-сервисов, которые обращаются в dadata, то для контроля запросов в dadata в одной точке, возможно потребуется прокси-кеш. Реализована воможность изменения дефолтных оригинальных url на url прокси-кеша

Velhron commented 3 years ago

Я планировал в ближайшее время доработать многие вещи, сделав вторую версию бандла. Спасибо за поддержку, я проверю ваши изменения.

@AlexandrErohin

AlexandrErohin commented 3 years ago

Запушил правку по codestyle @Velhron спасибо за ваш клиент. Хочу его использовать, но из-за инфраструктуры необходима возможность изменения оригинальных url на url прокси. Реализовал необязательную настройку через конфиг бандла - по умолчанию установка и api клиента не изменились. Буду признателен, если рассмотрите данную правку в рамках первой версии) Очень хочется воспользоваться вашим клиентом как можно скорее)

Velhron commented 3 years ago

Хорошо, я всё проверю и протестирую в ближайшие дни)

@AlexandrErohin

AlexandrErohin commented 3 years ago

@Velhron Спасибо)

AlexandrErohin commented 3 years ago

@Velhron Добрый день! Есть новости?)

Velhron commented 3 years ago

Добрый день. Принял ваши изменения. Сегодня опубликую новый релиз, необходимо добавить дополнительный функционал, который сделала ДаДата. Спасибо за помощь)

@AlexandrErohin