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

Add deprecation for not passing empty boxes option #6593

Merged
merged 1 commit into from
Nov 15, 2020

Conversation

VincentLanglet
Copy link
Member

@VincentLanglet VincentLanglet commented Nov 15, 2020

Subject

I am targeting this branch, because BC.

Related to #6589

Changelog

### Deprecated
- Not passing 'show', 'hide' or 'fade' as argument 4 to `Sonata\AdminBundle\Block\AdminSearchBlockService()`

franmomu
franmomu previously approved these changes Nov 15, 2020
@franmomu franmomu added the minor label Nov 15, 2020
@franmomu
Copy link
Member

An upgrade note is needed for the deprecation.

@VincentLanglet
Copy link
Member Author

An upgrade note is needed for the deprecation.

Done

UPGRADE-3.x.md Outdated Show resolved Hide resolved
src/Block/AdminSearchBlockService.php Outdated Show resolved Hide resolved
@VincentLanglet VincentLanglet force-pushed the emptyBoxesOption branch 2 times, most recently from a826411 to 9f90714 Compare November 15, 2020 13:19
@greg0ire greg0ire merged commit d181089 into sonata-project:3.x Nov 15, 2020
@greg0ire
Copy link
Contributor

Thanks @VincentLanglet ! Please refrain from marking conversations you did not started as resolved, it just makes it harder to see your replies, and it should really be the person that opens the conversation that decides if they are fine with what you did or not (you are obviously fine with what you do, or so I hope 😅 )

@VincentLanglet
Copy link
Member Author

Please refrain from marking conversations you did not started as resolved, it just makes it harder to see your replies, and it should really be the person that opens the conversation that decides if they are fine with what you did or not (you are obviously fine with what you do, or so I hope 😅 )

Oh ok, sorry ! No bad intentions ^^'

@VincentLanglet VincentLanglet deleted the emptyBoxesOption branch November 15, 2020 15:12
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.

3 participants