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: remove invalid props from dom #1532

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

VitaliyKosh
Copy link

Опишите проблему

PickerButton и TabsCollapsible при открытии вызывали ошибку передачи в DOM пропсов в camelCase

Шаги для воспроизведения

  1. Открыть компонент PickerButton в dev режиме
  2. Открыть компонент TabsCollapsible в dev режиме и нажать кнопку еще

Ожидаемое поведение

В консоли не должно быть ошибок

Чек лист

  • Тесты
  • Документация

Тестовый стенд

Copy link

changeset-bot bot commented Jan 15, 2025

🦋 Changeset detected

Latest commit: c334a18

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@alfalab/core-components-picker-button Patch
@alfalab/core-components-tabs Patch
@alfalab/core-components-custom-picker-button Patch

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

@coveralls
Copy link

Pull Request Test Coverage Report for Build 12786570123

Details

  • 2 of 2 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.001%) to 82.316%

Totals Coverage Status
Change from base Build 12708382287: 0.001%
Covered Lines: 10747
Relevant Lines: 11913

💛 - Coveralls

@core-ds-bot
Copy link
Collaborator

Собрана новая демка.

@denisx
Copy link
Contributor

denisx commented Jan 15, 2025

общий коммент к пр-ам такого плана: можно на основе таких исправлений искать подобные ошибки в других пакетах, и тоже делать подобные пры. как мейнейнеры, можно погружаться в такое (вижу 3-4 компонента с похожей проблемой)

@VitaliyKosh
Copy link
Author

общий коммент к пр-ам такого плана: можно на основе таких исправлений искать подобные ошибки в других пакетах, и тоже делать подобные пры. как мейнейнеры, можно погружаться в такое (вижу 3-4 компонента с похожей проблемой)

Проверю, еще остальные компоненты. Но пока поиск таких ошибок выглядит как открытие всех компонентов и перебор всех пропсов, в надежде на выпадение ошибок.

@VitaliyKosh VitaliyKosh marked this pull request as ready for review January 15, 2025 11:59
@denisx
Copy link
Contributor

denisx commented Jan 15, 2025

общий коммент к пр-ам такого плана: можно на основе таких исправлений искать подобные ошибки в других пакетах, и тоже делать подобные пры. как мейнейнеры, можно погружаться в такое (вижу 3-4 компонента с похожей проблемой)

Проверю, еще остальные компоненты. Но пока поиск таких ошибок выглядит как открытие всех компонентов и перебор всех пропсов, в надежде на выпадение ошибок.

ну это так, на светлое будущее. типа, чаще всего, если есть ошибка, то чаще рядом сидит ещё такое же ) виджетов-то больше сотки

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants