Skip to content

Conversation

xDeSwa
Copy link
Contributor

@xDeSwa xDeSwa commented Nov 14, 2024

Fix broken Link

Q A
Bug fix? yes
New feature? no
Issues Fix
License MIT

@carsonbot carsonbot added Bug Bug Fix Status: Needs Review Needs to be reviewed labels Nov 14, 2024
Copy link
Member

@Kocal Kocal left a comment

Choose a reason for hiding this comment

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

Thanks you for opening a pull-request, but the correction is not good.

In fact, what happens here is that https://github.com/symfony/ux/blob/2.x/ux.symfony.com/package.json existed, but it has been recently modified and moved to https://github.com/symfony/ux/blob/2.x/ux.symfony.com/assets/react/build/package.json in a refactoring PR (#2284 and another one IIRC).

Could you please update the link to the good file?

Thanks

@carsonbot carsonbot added Status: Needs Work Additional work is needed Status: Needs Review Needs to be reviewed and removed Status: Needs Review Needs to be reviewed Status: Needs Work Additional work is needed labels Nov 14, 2024
@carsonbot carsonbot added Status: Reviewed Has been reviewed by a maintainer and removed Status: Needs Review Needs to be reviewed labels Nov 14, 2024
@smnandre smnandre added docs Improvements or additions to documentation and removed Bug Bug Fix labels Nov 14, 2024
@smnandre smnandre changed the title Fix broken Link [React] Fix broken Link in docs Nov 14, 2024
@Kocal
Copy link
Member

Kocal commented Nov 17, 2024

Thank you @xDeSwa.

@Kocal Kocal merged commit eb8dd83 into symfony:2.x Nov 17, 2024
1 check passed
@xDeSwa xDeSwa deleted the patch-2 branch July 18, 2025 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation Status: Reviewed Has been reviewed by a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants