Skip to content

refactor(design-system)!: remove deprecated DsChip and DsChipGroup components [AR-53409]#341

Open
mmurawski-dn wants to merge 7 commits intodrivenets:nextfrom
mmurawski-dn:drivenets/michal/AR-53409-remove-ds-chip
Open

refactor(design-system)!: remove deprecated DsChip and DsChipGroup components [AR-53409]#341
mmurawski-dn wants to merge 7 commits intodrivenets:nextfrom
mmurawski-dn:drivenets/michal/AR-53409-remove-ds-chip

Conversation

@mmurawski-dn
Copy link
Copy Markdown
Collaborator

No description provided.

vpolessky-dn
vpolessky-dn previously approved these changes Apr 2, 2026
iromanchuk-dn
iromanchuk-dn previously approved these changes Apr 7, 2026
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

basically same comment as #340 (comment)

Comment on lines +93 to +99
ruleTester.run('no-deprecated-ds-date-input', plugin.rules['no-deprecated-ds-date-input'], {
valid: ['<DsDatePicker />', '<DsDateRangePicker />'],

invalid: [
{
code: '<DsChip />',
errors: [{ message: `DsChip is deprecated. Use DsTag instead.` }],
},
],
});

ruleTester.run('no-deprecated-ds-chip-group', plugin.rules['no-deprecated-ds-chip-group'], {
valid: ['<DsTag />', '<DsTagFilter />'],

invalid: [
{
code: '<DsChipGroup />',
errors: [{ message: `DsChipGroup is deprecated. Use DsTagFilter instead.` }],
code: '<DsDateInput />',
errors: [{ message: `DsDateInput is deprecated. Use DsDatePicker or DsDateRangePicker instead.` }],
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I guess this "date-input" diff is because next isn't updated from main?
if so, let's don't merge any PRs to next until we update it from main

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

next could have been updated and I would resolve merge conflicts. I think that should be straightforward to fix. @StyleShit do you know maybe can we add the "rebase/merge ff" option to the GH?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Removed this part, was my accident 👍

@mmurawski-dn mmurawski-dn force-pushed the drivenets/michal/AR-53409-remove-ds-chip branch 2 times, most recently from a85ee07 to dd850d5 Compare April 14, 2026 11:25
@mmurawski-dn mmurawski-dn force-pushed the drivenets/michal/AR-53409-remove-ds-chip branch from dd850d5 to 1d30c9d Compare April 14, 2026 11:27
StyleShit
StyleShit previously approved these changes Apr 14, 2026
vpolessky-dn
vpolessky-dn previously approved these changes Apr 14, 2026
@mmurawski-dn mmurawski-dn dismissed stale reviews from vpolessky-dn and StyleShit via c14b07e April 14, 2026 12:23
@StyleShit StyleShit changed the title refactor(design-system): remove deprecated DsChip and DsChipGroup components [AR-53409] refactor(design-system)!: remove deprecated DsChip and DsChipGroup components [AR-53409] Apr 15, 2026
…:mmurawski-dn/design-system into drivenets/michal/AR-53409-remove-ds-chip
@mmurawski-dn mmurawski-dn force-pushed the drivenets/michal/AR-53409-remove-ds-chip branch from 84633a3 to ec7209f Compare April 15, 2026 08:53
@mmurawski-dn mmurawski-dn force-pushed the drivenets/michal/AR-53409-remove-ds-chip branch from ec7209f to 5254586 Compare April 15, 2026 08:57
iromanchuk-dn
iromanchuk-dn previously approved these changes Apr 15, 2026
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.

4 participants