Skip to content

Conversation

@wouterj
Copy link
Member

@wouterj wouterj commented Jan 31, 2023

This adds a little section to the Security upgrade article, to make sure users are aware of this side-effect.

Note to merger: this is the latest version of this article - we removed it in 5.3. We should only merge this version into 5.3 and not do the merge-up after it.

@carsonbot

This comment was marked as resolved.

@javiereguiluz
Copy link
Member

Thanks Wouter.

@javiereguiluz javiereguiluz merged commit 0a32382 into symfony:5.2 Feb 2, 2023
@javiereguiluz
Copy link
Member

I think there's a typo in the original description because it says that we removed this doc in 5.3 but that we must merge this into 5.3.

That's why I merged this only on 5.2 branch.

@wouterj wouterj deleted the new-security-access-control branch February 2, 2023 18:09
@wouterj
Copy link
Member Author

wouterj commented Feb 2, 2023

Oh yes, only 5.2 is correct. Thanks for the merge Javier!

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.

3 participants