Skip to content

fix(MaskedInput): use @skbkontur/react-imask - #3605

Open
lossir wants to merge 5 commits into
nextfrom
IF-2008-imask-types
Open

fix(MaskedInput): use @skbkontur/react-imask#3605
lossir wants to merge 5 commits into
nextfrom
IF-2008-imask-types

Conversation

@lossir

@lossir lossir commented Feb 21, 2025

Copy link
Copy Markdown
Member

Проблема

Пакет react-imask криво собирается и ломает строгий резолвинг для cjs или esm. Из-за этого некоторые проекты не могут установить наш react-ui начиная с 5.0.

Обращение в исходный пакет было проигнорировано
uNmAnNeR/imaskjs#1050 (comment)

Также последний релиз исходного пакет был почти год назад
https://github.com/uNmAnNeR/imaskjs/tags

Решение

В итоге решил, что проще всего будет форкнуть репу, и самостоятельно исправить все проблемы.
https://git.skbkontur.ru/ui/imaskjs/-/merge_requests/2
https://www.npmjs.com/package/@skbkontur/react-imask

пакет форк тесты на проблемных проектах
5.0.12-fix-imask.2 @skbkontur/react-imask
5.0.8-fix-imask.1 @lossir/react-imask

Ссылки

Чек-лист перед запросом ревью

  1. Добавлены тесты на все изменения
    ⬜ unit-тесты для логики
    ⬜ скриншоты для верстки и кросс-браузерности
    ✅ нерелевантно

  2. Добавлена (обновлена) документация
    ⬜ styleguidist для пропов и примеров использования компонентов
    ⬜ jsdoc для утилит и хелперов
    ⬜ комментарии для неочевидных мест в коде
    ⬜ прочие инструкции (README.md, contributing.md и др.)
    ✅ нерелевантно

  3. Изменения корректно типизированы
    ⬜ без использования any (см. PR 2856)
    ✅ нерелевантно

  4. Прочее
    ✅ все тесты и линтеры на CI проходят
    ✅ в коде нет лишних изменений
    ✅ заголовок PR кратко и доступно отражает суть изменений (он попадет в changelog)

@lossir lossir closed this Mar 20, 2025
@lossir
lossir force-pushed the IF-2008-imask-types branch from 9e6046f to 6aeec26 Compare March 20, 2025 05:05
@lossir lossir reopened this Mar 20, 2025
@lossir
lossir marked this pull request as ready for review April 2, 2025 03:48
@lossir lossir changed the title If 2008 imask types fix(MaskedInput): use @skbkontur/react-imask Apr 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant