Skip to content

Conversation

dominikhajduk
Copy link
Contributor

PR after issue #7348 reported by @olberger.
Improved MicroKernel advanced example. Mentioned that build-in web server need to be run in web/ directory.

Copy link
Member

@javiereguiluz javiereguiluz left a comment

Choose a reason for hiding this comment

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

👍

Then see webpage in browser:

> http://localhost:8000/random/10
Copy link
Member

Choose a reason for hiding this comment

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

This is markdown syntax. I realize this isn't your fault, as it's wrong above as well. Can you please update both lines to be something like:

Then visit the webpage in the browser: http://localhost:8000/random/10

(actually, I really like to make these links clickable, but not sure what others think about that, so let's not do that in this PR)

@dominikhajduk
Copy link
Contributor Author

dominikhajduk commented Mar 5, 2017

Thanks @wouterj. Updated both places and squashed to one commit.

@wouterj
Copy link
Member

wouterj commented Mar 7, 2017

Thanks for your PR and quick actions Dominik!

@wouterj wouterj merged commit 6e9bb73 into symfony:2.8 Mar 7, 2017
wouterj added a commit that referenced this pull request Mar 7, 2017
…nikhajduk) This PR was merged into the 2.8 branch. Discussion ---------- Add MicroKernelTrait advanced example run section PR after [issue #7348](#7348) reported by @olberger. Improved [MicroKernel advanced example](http://symfony.com/doc/current/configuration/micro_kernel_trait.html#advanced-example-twig-annotations-and-the-web-debug-toolbar). Mentioned that build-in web server need to be run in `web/` directory. Commits ------- 6e9bb73 Add MicroKernelTrait advanced example run section
@dominikhajduk dominikhajduk deleted the patch-5 branch March 7, 2017 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

4 participants