8000 Add a missing comment for HTTP 451 status code by abdounikarim · Pull Request #46858 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

Add a missing comment for HTTP 451 status code #46858

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

Closed
wants to merge 0 commits into from

Conversation

abdounikarim
Copy link
Contributor

Hello,

I just added a missing comment for HTTP 451 status code to the associated RFC.

Q A
Branch? 5.4
Bug fix? no
New feature? no
Deprecations? no
Tickets Fix #...
License MIT
Doc PR symfony/symfony-docs#...
I added RFC7725 for HTTP 451 status code.

@carsonbot carsonbot added this to the 5.4 milestone Jul 5, 2022
@fabpot fabpot modified the milestones: 5.4, 4.4 Jul 5, 2022
@fabpot fabpot changed the base branch from 5.4 to 4.4 July 5, 2022 17:41
@fabpot
Copy link
Member
fabpot commented Jul 5, 2022

Thank you @abdounikarim.

fabpot added a commit that referenced this pull request Jul 5, 2022
…arim)

This PR was submitted for the 5.4 branch but it was merged into the 4.4 branch instead.

Discussion
----------

Add a missing comment for HTTP 451 status code

Hello,

I just added a missing comment for HTTP 451 status code to the associated RFC.

| Q             | A
| ------------- | ---
| Branch?       | 5.4
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix #...
| License       | MIT
| Doc PR        | symfony/symfony-docs#...
I added RFC7725 for HTTP 451 status code.

Commits
-------

e64c015 Update Response.php
@fabpot fabpot closed this Jul 5, 2022
@abdounikarim abdounikarim deleted the patch-2 branch July 6, 2022 12:48
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
0