Repository navigation
Ibexa Doctrine Migrations - #3337
adriendupuis wants to merge 27 commits into
Conversation
Summary
Errors per inputErrors in site/api/php_api/php_api/index.html
Errors in site/cdp/cdp_activation/cdp_data_export/index.html
Errors in site/content_management/field_types/field_type_reference/urlfield/index.html
Errors in site/getting_started/requirements/index.html
Errors in site/ibexa_cloud/ibexa_cloud_guide/index.html
Errors in site/ibexa_cloud/install_on_ibexa_cloud/index.html
Errors in site/ibexa_products/ibexa_commerce/index.html
Errors in site/ibexa_products/ibexa_experience/index.html
Errors in site/ibexa_products/ibexa_headless/index.html
Errors in site/infrastructure_and_maintenance/cache/http_cache/reverse_proxy/index.html
Errors in site/infrastructure_and_maintenance/security/reporting_issues/index.html
Errors in site/infrastructure_and_maintenance/security/security_checklist/index.html
Errors in site/permissions/custom_policies/index.html
Errors in site/product_catalog/quable/install_quable/index.html
Errors in site/product_catalog/quable/quable_api/index.html
Errors in site/recommendations/raptor_integration/connector_installation_configuration/index.html
Errors in site/recommendations/raptor_integration/raptor_connector/index.html
Errors in site/recommendations/raptor_integration/raptor_connector_guide/index.html
Errors in site/release_notes/cohesivo_v6.0_deprecations/index.html
Errors in site/release_notes/ez_platform_v1.10.0/index.html
Errors in site/release_notes/ez_platform_v1.11.0/index.html
Errors in site/release_notes/ez_platform_v1.12.0/index.html
Errors in site/release_notes/ez_platform_v1.13.0_lts/index.html
Errors in site/release_notes/ez_platform_v1.7.0_lts/index.html
Errors in site/release_notes/ez_platform_v1.8.0/index.html
Errors in site/release_notes/ez_platform_v1.9.0/index.html
Errors in site/release_notes/ibexa_dxp_v4.0/index.html
Errors in site/release_notes/ibexa_dxp_v4.4/index.html
Errors in site/release_notes/ibexa_dxp_v4.6/index.html
Errors in site/release_notes/ibexa_dxp_v5.0/index.html
Errors in site/resources/new_in_doc/index.html
Errors in site/update_and_migration/from_4.5/update_from_4.5/index.html
Redirects per inputRedirects in site/administration/project_organization/bundles/index.htmlRedirects in site/api/php_api/php_api/index.html
Redirects in site/cdp/cdp_activation/cdp_data_export/index.htmlRedirects in site/commerce/transactional_emails/transactional_emails/index.htmlRedirects in site/content_management/field_types/field_type_reference/urlfield/index.html
Redirects in site/getting_started/requirements/index.html
Redirects in site/ibexa_products/ibexa_commerce/index.html
Redirects in site/ibexa_products/ibexa_experience/index.html
Redirects in site/ibexa_products/ibexa_headless/index.html
Redirects in site/infrastructure_and_maintenance/cache/http_cache/reverse_proxy/index.html
Redirects in site/infrastructure_and_maintenance/security/reporting_issues/index.html
Redirects in site/product_catalog/quable/install_quable/index.html
Redirects in site/product_catalog/quable/quable/index.html
Redirects in site/release_notes/cohesivo_v6.0_deprecations/index.html
Redirects in site/release_notes/ez_platform_v1.12.0/index.htmlRedirects in site/release_notes/ibexa_dxp_v4.0/index.html
Redirects in site/release_notes/ibexa_dxp_v4.4/index.html
Redirects in site/release_notes/ibexa_dxp_v5.0/index.html
Redirects in site/resources/new_in_doc/index.html
Redirects in site/templating/components/index.html
Redirects in site/templating/image_variations/index.html |
strict-booleans rule set is removed in Rector 2.6 https://github.com/rectorphp/rector/releases/tag/2.6.0
continue 1b4038a
There was a problem hiding this comment.
Warning
Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.
Copilot review overview
7 open issues
The admonition syntax appears malformed (missing opening quote before the title and an extra quote… · New As written, this looks like two commands executed in the same non-interactive shell, but… · New The heading#### Modify database schemais duplicated, which will render oddly and can confuse… · New There are two consecutive admonitions with the same titleCommand scope, but the second one… · New There are two consecutive admonitions with the same titleCommand scope, but the second one… · New Correct spelling of 'cnfiguration' to 'configuration'. · New This line contains trailing whitespace at the end, which can cause style/vale/linters to fail in… · New
What changed in this PR
Updates Ibexa documentation to reflect the introduction of Ibexa Doctrine Migrations for managing schema upgrades instead of manual SQL scripts.
Changes:
- Replace SQL upgrade/script instructions with
ibexa:doctrine:migrations:*commands across installation and update docs. - Add new versioned upgrade notes for v4.6.33 and v5.0.11 explaining how to run migrations safely.
- Update the 4.6 → 5.0 upgrade guide to reference Symfony 7.4 and adjust database-upgrade guidance.
| File | Description |
|---|---|
| docs/users/oauth_server.md | Switch OAuth2 server schema setup from SQL dump pipelines to Ibexa Doctrine migrations. |
| docs/update_and_migration/from_5.0/update_from_5.0.md | Add v5.0.11 section documenting Doctrine migrations-based DB updates. |
| docs/update_and_migration/from_4.6/update_to_5.0.md | Update 4.6→5.0 guide: migrations-based DB upgrade, Symfony 7.4 requirement, and revised conclusions. |
| docs/update_and_migration/from_4.6/update_from_4.6.md | Add v4.6.33 section documenting Doctrine migrations-based DB updates. |
| docs/multisite/translations_management/configure_translations_management.md | Replace SQL schema snippets with migration status/migrate commands. |
| docs/ibexa_cloud/ibexa_cloud_cli.md | Add Cloud CLI example for running migrations over SSH; minor wording tweak. |
| docs/commerce/shopping_list/install_shopping_list.md | Replace SQL schema snippets with migration status/migrate commands for Shopping List. |
Review effort: Lite
…uire reversing if changes are not merged in time)
Steveb-p
left a comment
There was a problem hiding this comment.
Thanks, most of the earlier points are in. Remaining:
- Unreferenced files - remove?
code_samples/shopping_list/install/schema.*.sqlcode_samples/translations_management/install/schema.*.sqlfrom_4.6/sql/ibexa_oss_4.6.latest-to-5.0.0_*.sql
- v4.6.33 is only in the 5.0 docs - is a 4.6 counterpart planned?
- The "Preview only" caution is right for now. If ibexa/doctrine-migrations#25-#27 land in 5.0.11, we'll revisit it.
- In shopping list and translations management, the caution snippet splits the bullet list.
Steveb-p
left a comment
There was a problem hiding this comment.
I believe we should also include a note somewhere on how one can apply each migration in sequence (in lieu of running all of them all at once).
php bin/console ibexa:doctrine:migrations:migrate next --dry-run...and without --dry-run can allow user to go through each migration, one by one.
|
|
||
| !!! note "Installing with Doctrine migrations" | ||
|
|
||
| By default, `ibexa:install` creates the database schema directly, without tracking individual changes. |
There was a problem hiding this comment.
Side note to memory: in 6.0, installing via migrations will be the default.



ibexa/doctrine-migrations#1
Thiis PR will be upmerged to 6.0 but we must remember to adjust 6.0 description to match the final shape of the feature in that branch.
Wait with merge: Paweł is making more changes to the feature related to the dry-run procedure. Wait with implementing his commentss until he confirms the feature behavior is as required.
Checklist