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/file-upload): emit events for clear files fix #1848 #1945

Merged
merged 1 commit into from
Aug 12, 2024

Conversation

ickisIckis
Copy link
Collaborator

fix(components/file-upload): emit events for clear files fix #1848

Библиотека

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

Компонент

FileUpload

Задача

resolved #1858

Изменения

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

Checklist:

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

Следует обратить внимание при тестировании

Способ проверки: multiply = false, добавить один файл, затем заменить его другим. Cобытия beforeFilesChange и аilesChange должно сработать 1 раз, на проде сейчас дублируются из-за ошибки в логике.

Release Notes

Исправили ошибку в поведении функции ClearFiles c опцией emitEvent: false

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

@ickisIckis ickisIckis added the state:ready ready for release (pass tested) label Aug 12, 2024
@ickisIckis ickisIckis merged commit fffa7b8 into v3 Aug 12, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants