Repository navigation
Ibexa Doctrine Migrations - #3337
Open
adriendupuis wants to merge 20 commits into
Open
adriendupuis wants to merge 20 commits into
adriendupuis wants to merge 20 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_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/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_3.3/update_from_3.3/index.html
Errors in site/update_and_migration/from_4.6/update_from_4.6/index.html
Redirects per inputRedirects in site/ai/mcp/mcp_usage/index.html
Redirects in site/api/php_api/php_api/index.html
Redirects 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_cloud/ibexa_cloud_guide/index.htmlRedirects 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/quable_api/index.html
Redirects in site/recommendations/raptor_integration/connector_installation_configuration/index.htmlRedirects in site/recommendations/raptor_integration/raptor_connector/index.htmlRedirects in site/recommendations/raptor_integration/raptor_connector_guide/index.htmlRedirects in site/release_notes/cohesivo_v6.0_deprecations/index.html
Redirects in site/release_notes/ez_platform_v1.9.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
|
strict-booleans rule set is removed in Rector 2.6 https://github.com/rectorphp/rector/releases/tag/2.6.0
continue 1b4038a
dabrt
marked this pull request as ready for review
October 19, 2026 13:57
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
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



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.
Checklist