-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Add docs for the new WebLink component #7515
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
Comments
This feature is now a new component called WebLink. See symfony/symfony#22273 and https://symfony.com/blog/new-in-symfony-3-3-weblink-component |
Would it be possible to mention this in "http://symfony.com/blog/new-in-symfony-3-3-asset-preloading-with-http-2-push"? Google finds that when searching for "http/2 push symfony". Also, a quick mention of how to enable it. |
@dunglas has published a nice article about WebLink and HTTP/2 push, etc. See https://dunglas.fr/2017/10/symfony-4-http2-push-and-preloading/ He asked me if it would be OK to use that article as the base of the docs of the WebLink component. I think it's a good idea. @xabbuh @wouterj @weaverryan what do you think? Thanks! |
I think it should be split into several smaller articles or shortened a bit, but as a starting point this looks like a good idea to me. |
This PR was merged into the 3.4 branch. Discussion ---------- Add docs for the WebLink component The WebLink component is available since Symfony 3.3, but I never took the time to add the docs (however, a blog post explaining how to use it was available). This documentation is based on https://dunglas.fr/2017/10/symfony-4-http2-push-and-preloading/. If necessary, I can grant any copyright regarding this post to the Symfony project. symfony/symfony#21478 symfony/symfony#22273 Closes #7515. Commits ------- 91ee3bc Fix RST ea7b3da @nicolas-grekas' review 38fda88 fix build e12e776 RST 088690f Fix link e3d4036 RST 178821e refactor 9f4ae9b fix typo 6beb4eb Add docs for the WebLink component
Fixed by #10309. |
see symfony/symfony#21478
The text was updated successfully, but these errors were encountered: