Skip to content

[AssetMapper] Removing duplication #19629

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

Merged
merged 1 commit into from
Mar 4, 2024

Conversation

ThomasLandauer
Copy link
Contributor

Page: https://symfony.com/doc/6.4/frontend/asset_mapper.html#importing-3rd-party-javascript-packages

Reason: The command was in the text and then again in the code block.

@carsonbot carsonbot added this to the 6.4 milestone Mar 4, 2024
@javiereguiluz javiereguiluz merged commit 28e6757 into symfony:6.4 Mar 4, 2024
@javiereguiluz
Copy link
Member

Thomas, I agree on removing the duplicated command in the text. That makes docs easier to maintain.

However, I don't fully agree with the proposed text and had to revert part of it while merging (See 4f253a5).

The problem is using the wording "To intialize your project..." because you need to run that command not only when initializing the project, but whenever someone adds new assets to the project and you fetch the changes from the repo.

@ThomasLandauer ThomasLandauer deleted the patch-25 branch March 4, 2024 16:20
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.

3 participants