Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add information to sub request #16561

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
Jan 18, 2024
Merged

Conversation

DamienDeSousa
Copy link
Contributor

@DamienDeSousa DamienDeSousa commented Mar 3, 2022

Adding some information about _format attribute default value because I spend some times to understand why this wasn't working for me. Now I know why it was not working, _format value was html by default and I get no error message. So I just share with orther developers this little update to avoid them spending the same time.

Thank you, have a nice day :)

@carsonbot carsonbot added this to the 4.4 milestone Mar 3, 2022
@DamienDeSousa DamienDeSousa changed the title Update http_kernel.rst Update http_kernel.rst, add information to sub request Mar 4, 2022
@@ -742,6 +742,11 @@ look like this::

.. _http-kernel-resource-locator:

Also, note that by default, if the ``_format`` attribute is not set in your request, the value will be ``html``.
Copy link
Contributor

Choose a reason for hiding this comment

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

is this the best part to add such info notice for such _format ?

@OskarStark OskarStark changed the title Update http_kernel.rst, add information to sub request Add information to sub request Nov 9, 2022
@@ -742,6 +742,11 @@ look like this::

.. _http-kernel-resource-locator:
Copy link
Contributor

Choose a reason for hiding this comment

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

this must be moved above the next Headline Locating Resources again

Copy link
Member

Choose a reason for hiding this comment

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

Done while merging. Thanks.

@javiereguiluz javiereguiluz changed the base branch from 4.4 to 5.4 January 18, 2024 12:04
@javiereguiluz javiereguiluz merged commit 6bab099 into symfony:5.4 Jan 18, 2024
@javiereguiluz
Copy link
Member

We finally merged this. Thanks a lot Damien ... and congrats on your first Symfony Docs contribution 🎉

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.

6 participants