Skip to content

Drop support for sonata-project/block-bundle 4#8344

Merged
VincentLanglet merged 1 commit intosonata-project:4.xfrom
dmaicher:drop_block_bundle_4
Oct 29, 2025
Merged

Drop support for sonata-project/block-bundle 4#8344
VincentLanglet merged 1 commit intosonata-project:4.xfrom
dmaicher:drop_block_bundle_4

Conversation

@dmaicher
Copy link
Copy Markdown
Contributor

Subject

I am targeting this branch, because its BC.

Let's drop support for v4 to simplify things. Will also look into dropping support for other unmaintained sonata package versions as well.

Changelog

### Removed
- support for `sonata-project/block-bundle` v4 was removed

@dmaicher dmaicher force-pushed the drop_block_bundle_4 branch from d7c390b to 5a034a3 Compare October 29, 2025 07:49
@dmaicher dmaicher requested a review from a team October 29, 2025 07:53
@dmaicher dmaicher marked this pull request as ready for review October 29, 2025 07:54

$blockExtension = new SonataBlockExtension();
/*
* TODO: remove "http_cache" parameter when support for SonataBlockBundle 4 is dropped.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure to understand the comment @dmaicher, shouldn't we keep

$blockExtension->load(['sonata_block' => []], $container);

or it's a useless call ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well tests are passing 😄 So I think it was a useless call. No need to load the extension. This was just loaded to prevent a deprecation I believe.

@VincentLanglet VincentLanglet merged commit c370dbf into sonata-project:4.x Oct 29, 2025
24 checks passed
@dmaicher dmaicher deleted the drop_block_bundle_4 branch October 29, 2025 09:50
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.

2 participants