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.

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

Fixed #14186, fixed #14510 - forced validation regression (caused by #13105 fix) #14554

Merged
merged 2 commits into from
Jul 29, 2017

Conversation

arogachev
Copy link
Contributor

@arogachev arogachev commented Jul 29, 2017

Q A
Is bugfix? yes
New feature? no
Breaks BC? no
Tests pass? yes
Fixed issues #14186, #14510
Related issues #13105

There is a regression in forced validation caused by @SilverFire's commit 7d494c1 (which is a fix for #13105). I reverted the changes, these two issues (#14186, #14510) should be fixed now. I'll reopen #13105 for better fix with tests.

@arogachev
Copy link
Contributor Author

arogachev commented Jul 29, 2017

I configured GPG key, but forgot to set up signing commits using it for Yii 2 repo. Sorry for that. 😞 Will do for future commits. 😄

@samdark samdark merged commit 8f2d9ba into master Jul 29, 2017
@samdark
Copy link
Member

samdark commented Jul 29, 2017

Merged. Thank you!

@samdark samdark deleted the 14186-14510-force-validate-regression branch July 29, 2017 11:37
@samdark samdark self-assigned this Jul 29, 2017
@samdark samdark added the type:bug Bug label Jul 29, 2017
@samdark samdark added this to the 2.0.13 milestone Jul 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants