Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

[SecurityBundle] Create a smooth upgrade path for security factories#41754

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

@wouterj
Copy link
Member

QA
Branch?5.4
Bug fix?no
New feature?yes
Deprecations?yes
TicketsRef#41613 (comment)
LicenseMIT
Doc PR-

This change allows removingSecurityFactoryInterface in Symfony 6.

I've also changed the discrete ordering using "listener positions" to the much more common continuous ordering using priorities. I feel like priorities are much more self-explanatory.

chalasr reacted with thumbs up emoji
@nicolas-grekasnicolas-grekas added this to the5.4 milestoneJun 21, 2021
@wouterjwouterjforce-pushed thepull-41613/authenticator-factory branch fromfc667f4 to3a5ac34CompareJune 27, 2021 10:40
@StaffNowa
Copy link
Contributor

Please make fabbot happy 😊

sstok reacted with laugh emoji

@wouterjwouterjforce-pushed thepull-41613/authenticator-factory branch 2 times, most recently frome81ff82 to5b54abcCompareJuly 1, 2021 19:19
@wouterj
Copy link
MemberAuthor

This is as green as this PR will get. The remaining errors are false positives.

@wouterjwouterjforce-pushed thepull-41613/authenticator-factory branch from0172e56 to9c44ad6CompareJuly 3, 2021 21:21
@chalasr
Copy link
Member

small rebase needed

@wouterjwouterjforce-pushed thepull-41613/authenticator-factory branch from9c44ad6 to7385fd5CompareAugust 7, 2021 15:12
@wouterj
Copy link
MemberAuthor

Sorry for the delay, I forgot this one was still open. Rebased and ready :)

@chalasr
Copy link
Member

Thanks Wouter.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@chalasrchalasrchalasr left review comments

@fabpotfabpotfabpot requested changes

+1 more reviewer

@abunchabunchabunch left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Projects

None yet

Milestone

5.4

Development

Successfully merging this pull request may close these issues.

7 participants

@wouterj@StaffNowa@chalasr@fabpot@abunch@nicolas-grekas@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp