Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix a method name in the TwoFactorFrontendListenerTest class #1131

Merged
merged 1 commit into from
Dec 23, 2019
Merged

Fix a method name in the TwoFactorFrontendListenerTest class #1131

merged 1 commit into from
Dec 23, 2019

Conversation

bytehead
Copy link
Member

@bytehead bytehead commented Dec 23, 2019 •
edited

Changed method signature in #1125.

@bytehead bytehead self-assigned this Dec 23, 2019
@bytehead bytehead added this to the 4.9 milestone Dec 23, 2019
@bytehead bytehead added the bug label Dec 23, 2019
@leofeyer leofeyer merged commit df231e7 into contao:master Dec 23, 2019
@leofeyer
Copy link
Member

Thank you @bytehead.

leofeyer added a commit that referenced this pull request Dec 23, 2019
@leofeyer
Revert "Renamed method after changing its signature (see #1131)"
732b7b0 
This reverts commit df231e7.
@leofeyer leofeyer changed the title Renamed method after changing its signature Renam method after changing its signature Dec 23, 2019
@leofeyer leofeyer changed the title Renam method after changing its signature Rename method after changing its signature Dec 23, 2019
@bytehead bytehead deleted the bugfix/2fa-tests-renaming-method branch January 5, 2020 19:07
@leofeyer leofeyer changed the title Rename method after changing its signature Fix a method name in the TwoFactorFrontendListenerTest class Jan 10, 2020
@leofeyer leofeyer added CI and removed CI labels Jan 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leofeyer leofeyer leofeyer approved these changes

Assignees

@bytehead bytehead

Labels
bug
Projects
None yet
Milestone
4.9
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@bytehead @leofeyer