Skip to content

Conversation

@alexachel
Copy link
Contributor

@alexachel alexachel commented Jan 15, 2026

К кнопкам действий добавлен атрибут data-test-id

Чек лист

  • Задача сформулирована и описана в JIRA
  • В названии ветки есть айдишник задачи в JIRA (fix/DS-1234), ссылку прикреплять не надо
  • У реквеста осмысленное название feat(...) или fix(...) по conventional commits (https://www.conventionalcommits.org)
  • Код покрыт тестами и протестирован в различных браузерах
  • Добавленные пропсы добавлены в демки и описаны в документации
  • К реквесту добавлен changeset

Если есть визуальные изменения

  • Прикреплено изображение было/стало

@changeset-bot
Copy link

changeset-bot bot commented Jan 15, 2026

🦋 Changeset detected

Latest commit: d2b8b76

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

This PR includes changesets to release 2 packages
Name Type
@alfalab/core-components-file-upload-item Patch
@alfalab/core-components 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

coveralls commented Jan 15, 2026

Pull Request Test Coverage Report for Build 21059294287

Details

  • 5 of 5 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.003%) to 79.861%

Totals Coverage Status
Change from base Build 20998327799: 0.003%
Covered Lines: 9725
Relevant Lines: 11412

💛 - Coveralls

@github-actions
Copy link
Contributor

Demo build

https://core-ds.github.io/core-components/2013

Copy link
Contributor

@fulcanellee fulcanellee left a comment

Choose a reason for hiding this comment

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

Нужно немного поправить. По аналогии с другими компонентами, добавить функцию которая позволит более удобно получить элементы для тестов снаружи библиотеки

https://github.com/core-ds/core-components/blob/master/packages/button/src/utils.ts#L3

Реэкспорт через shared
https://github.com/core-ds/core-components/blob/master/packages/button/src/shared/index.ts#L1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants