appwilio/cdek-sdk

Открытие ветки beta-common-integration

Closed this issue · 7 comments

volga commented

В связи с наличием beta версии common-integration у СДЭК - предлагаю открыть новую ветку разработки beta-common-integration. СДЭК предоставил мне доступ к ней. Как таковой документации там еще не существует.

Первая необходимая правка в новой ветке:

  • В тег ордер необходимо добавить данную информацию для изменения отправителя на третье лицо
<Sender company="РУ_ЕКАТЕРИНБУРГ_Мега_BCS" name="ООО Адидас">
            <address flat=" " house="-" street="620131, Екатеринбург, Металлургов,87  "/>
            <phone>8-800-250-04-05</phone>
</Sender>

Также в новой версии отличаются ответы.

Готов сделать первые правки для интеграции с бета версией в новой ветке.

Можно хоть где-то почитать об отличиях?

volga commented

Нет, почитать негде - в том то и дело.
Разработчики СДЭК мне сообщили на словах,, что ответы отличаются, документации нет.
Первое отличие в моем запросе - тег Sender.

Я, конечно, могу сделать форк и начать параллельную разработку - но хотелось бы делать это в одной точке.

@volga делайте PR в мастер, я его потом в отдельную ветку замержу.

volga commented

#8 - @jhaoda, сделал

@volga документация по api у них убогая. Большинство вещей делались "на ощупь".
Скорее всего, где-то мне попалась дата именно в таком формате, и я по наивности своей решил, что у них единый формат дат. А может и не так это было - полтора года прошло.
Кстати, не удивлюсь если они поменяли формат и никому не сказали. За ними водится такое.

@jhaoda открой товарищу репу уже, пусть сделает ветку и пилит в нее. Что мы будем тут как индейцы с пулл-реквестами скакать? Тем более, что ни у меня ни у тебя полноценно занятся этим времени нет. Мужик адекватный же.

volga commented

Спасибо! Запрос закрываю.

@greabock ну шо ты, я просто заэтался 😉 Создал ветку, добавил в список доверенных.