fix(intl-phone-input): ability to overwrite country-data #1562
Conversation
mikhail-yurin
requested review from
reme3d2y,
fulcanellee,
SiebenSieben and
hextion
as code owners
🦋 Changeset detectedLatest commit: 0eeb397 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Pull Request Test Coverage Report for Build 13135311953Details
💛 - Coveralls |
| /** | ||
| * | ||
| * Список правил парсинга номеров телефонов по странам (для переопределения дефолтного) | ||
| * @type {CountriesData[]} |
There was a problem hiding this comment.
Пожалуй это лишнее (@type {CountriesData[]}), ведь мы находимся в парадигме ts и переданный тип для пропса говорит сам за себя.
There was a problem hiding this comment.
Это описание (вместе с типом) нужно для сторибука, благодаря этому doc отображается параметр с его описанием
There was a problem hiding this comment.
я имел ввиду только строчку 50, где указан @type. описание трогать не нужно
There was a problem hiding this comment.
убрал @type {CountriesData[]}
|
|
Собрана новая демка. |
|
|
Собрана новая демка. |
.changeset/violet-dancers-check.md
Outdated
| '@alfalab/core-components-international-phone-input': minor | ||
| --- | ||
|
|
||
| optional prop customCountriesList was added to be able to overwrite default country-data |
There was a problem hiding this comment.
Напиши на русском, пожалуйста, это описание попадает в ченджлог
|
Собрана новая демка. |
* fix(intl-phone-input): ability to overwrite country-data * fix(intl-phone-input): changes after review * fix(intl-phone-input): fix after review --------- Co-authored-by: Юрин Михаил Михайлович <[email protected]> (cherry picked from commit 0e8f85b)
Опишите проблему
Требование от НСПК: возможная длина номера тлф до 15 цифр
Добавил пропс для переопределения дефолтного массива countriesData, чтобы по месту использования компонента можно было передать массив в котором уже будет нужная длина номеров телефонов