Skip to content

Conversation

@ker0x
Copy link
Contributor

@ker0x ker0x commented Nov 12, 2025

Q A
Bug fix? yes
New feature? no
Deprecations? no
Documentation? no
Issues -
License MIT

@ker0x ker0x requested a review from Kocal as a code owner November 12, 2025 16:48
@carsonbot carsonbot added Bug Bug Fix Status: Needs Review Needs to be reviewed labels Nov 12, 2025
@carsonbot carsonbot changed the title Fix/shadcn button link example Fix/shadcn button link example Nov 12, 2025
@ker0x ker0x changed the title Fix/shadcn button link example [Toolkit][Shadcn] Add missing as props, update CONTRIBUTING.md Nov 12, 2025
@ker0x ker0x force-pushed the fix/shadcn-button-link-example branch 2 times, most recently from e42d191 to be4b9a8 Compare November 12, 2025 16:55
@carsonbot carsonbot added Status: Reviewed Has been reviewed by a maintainer and removed Status: Needs Review Needs to be reviewed labels Nov 13, 2025
@carsonbot carsonbot changed the title [Toolkit][Shadcn] Add missing as props, update CONTRIBUTING.md [Shadcn] Add missing as props, update CONTRIBUTING.md Nov 13, 2025
@Kocal Kocal force-pushed the fix/shadcn-button-link-example branch from be4b9a8 to e971f96 Compare November 13, 2025 07:48
CONTRIBUTING.md Outdated
Comment on lines 80 to 91
#### Generate snapshots for UX Toolkit

If you make changes to a component template or add a new one, create or update the EXAMPLES.md of the component.
Then, you can (re)generate new snapshots by running:

```shell
php vendor/bin/simple-phpunit -d --update-snapshots
```

> [!IMPORTANT]
> Snapshots will not reflect changes if you forget to update the EXAMPLES.md.
Copy link
Member

Choose a reason for hiding this comment

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

This section is already present in the Toolkit/CONTRIBUTING.md file

[Toolkit][Shadcn] Add missing as props, update CONTRIBUTING.md
@Kocal Kocal force-pushed the fix/shadcn-button-link-example branch from e971f96 to 56a8a90 Compare November 13, 2025 07:49
@Kocal
Copy link
Member

Kocal commented Nov 13, 2025

Thank you @ker0x.

@Kocal Kocal merged commit 766d9f8 into symfony:2.x Nov 13, 2025
28 of 29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Bug Fix Status: Reviewed Has been reviewed by a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants