Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(components/multiselect): usage with transformers should cover case with selectAll option #1919 #1943

Merged
merged 1 commit into from
Aug 9, 2024

Conversation

ickisIckis
Copy link
Collaborator

@ickisIckis ickisIckis commented Aug 8, 2024

fix(components/multiselect): usage with transformers should cover case with selectAll option #1919

Библиотека

  • @prizm-ui/components
  • @prizm-ui/install
  • @prizm-ui/icons
  • @prizm-ui/theme

Компонент

multiselect

Задача

resolved #1919

Изменения

  • Имеются BREAKING CHANGES
  • Изменения документации
  • Добавление фичи
  • Исправление бага

Checklist:

  • После фичи обновил документацию
  • Сделал код чище чем был до этого
  • Тесты и линтер на рабочей машине успешно выполнились

Release notes

Исправили ошибку обработки опции 'Выбарть все' при использовании трансформера в мультселекте.

Copy link

github-actions bot commented Aug 8, 2024

Visit the preview URL for this PR (updated for commit 6c32914):

https://prizm-v4--pr1943-fix-multiselect-tran-fq4u59lr.web.app

(expires Tue, 13 Aug 2024 10:20:04 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 7c62ed8dbabf5e2d6b2084ca9e107cc206d30dbd

Copy link
Contributor

@ZurabDev ZurabDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ZurabDev ZurabDev added the availability:reviewed Ready to test label Aug 9, 2024
@alexhawkins94
Copy link

При очистке инпута кнопкой очистки в value прокидывается null.

При последовательном удалении всех чипсов из инпута кнопкой закрытия чипса в value отображается пустой массив [].

Это корректно?
@ZurabDev @ickisIckis

chrome_OolwnaRmS5
chrome_8vNbofM2Ki

@ZurabDev ZurabDev merged commit 23d9a20 into main Aug 9, 2024
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants