Skip to content

Conversation

@alongosz
Copy link
Member

@alongosz alongosz commented Nov 25, 2024

Caution

This is a part of bigger set of changes, to be merged together when ready

  • Remove TMP commits before merge
🎫 Issue IBX-8470

Related PRs:

Description:

This PR bumps Symfony to v6 with necessary and optional codebase upgrades.

We had here a cyclic dependency on ibexa/user package. The deprecated Ibexa User extension points code was removed together with the dependency itself.

Key changes:

  • [Composer] Dropped ci-scripts from dev requirements
  • Bumped Symfony packages requirements to ^6.4
  • Dropped deprecated Ibexa User Bundle extension points
  • [Composer] Bumped jms/translation-bundle ^2.4 for Symfony 6 support
  • [Composer] Dropped obsolete dependency on ibexa/user
  • [Tests] Aligned form mapper tests with Symfony 6
  • Added strict types to Symfony Bundle extension points methods
  • Improved Symfony Bundle extension points code quality
  • Refactored Legacy naming for Ibexa Core extension
  • [Doc] Dropped obsolete doc directory

QA:

Sanities in the form of regression tests.

Documentation:

Dropped deprecated:

  • Ibexa\ContentForms\User\View\UserRegisterConfirmView in favor of Ibexa\User\View\UserRegisterConfirmView
  • Ibexa\ContentForms\User\View\UserRegisterFormView in favor for Ibexa\User\View\UserRegisterFormView
  • Ibexa\Bundle\ContentForms\Controller\UserRegisterController in favor of \Ibexa\Bundle\User\Controller\UserRegisterController

@alongosz alongosz added Doc needed The changes require some documentation Feature New feature request Ready for review labels Nov 25, 2024
@alongosz alongosz marked this pull request as ready for review November 25, 2024 13:36
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 4, 2025

@adamwojs adamwojs merged commit d127130 into main Feb 4, 2025
7 of 8 checks passed
@adamwojs adamwojs deleted the ibx-8470-symfony-6 branch February 4, 2025 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Doc needed The changes require some documentation Feature New feature request Ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants