add necessary key logic #580
Conversation
|
There was a problem hiding this comment.
Другая версия npm. А чего мы его вообще обновили?
There was a problem hiding this comment.
Конфликты порезолвил, а вот тесты надо будет пофиксить, там всетаки апи поменялись
There was a problem hiding this comment.
А вот эти контракты можно найти где-то?
There was a problem hiding this comment.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
There was a problem hiding this comment.
да , я поменял сейчас уже работает на каталог 3.0
There was a problem hiding this comment.
Это условие в будущем изменится?
Сейчас я вижу: нет ключа -> грузим по старому урлу.
There was a problem hiding this comment.
Нет, там фишка в том что потом когда закроют api ошибка в любом случаи будет кидаться и окно с блокировкой
Co-authored-by: Evgeny Rodygin <[email protected]>
| this._urlGeoGet = apiUrl + 'geo/get'; | ||
| this._urlDetails = apiUrl + 'catalog/branch/get'; | ||
| this._urlFirmsInHouse = apiUrl + 'catalog/branch/list'; | ||
| this._urlGeoSearch = apiUrl + 'items/geocode'; |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
Co-authored-by: Oleg Kalyanov <[email protected]>
|
@Farfurix @EvgenyRodygin вы финально будете смотреть, или аппрувы валидны? |
Нужно проверить код, плюс логику работы с ключом ,я проверял три кейса добавлял валидный ключ (ruregt3044) в данный момент он валиден для каталога и тайлов потом он будет работать только для region/list , пробовал без ключа, сейчас без ключа должно автоматически начать ходить на старое api и добавлял не валидный ключ, проверка загрузки тайлов (с ключом работает только под впн так как сейчас тестируем на stage после того как выкатим tileserver на прод можно будет проверить на проде) проверить клик и получения geosearch