Skip to content

Add MicroKernelTrait advanced example run section #7570

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 7, 2017

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants