Skip to content

Migrate and refactor data import/export - #18

Merged
javihgil merged 2 commits into
6.0from
migrate-import-export-features-to-new-cms-data-plugin
Mar 26, 2026
Merged

Migrate and refactor data import/export#18
javihgil merged 2 commits into
6.0from
migrate-import-export-features-to-new-cms-data-plugin

Conversation

@javihgil

Copy link
Copy Markdown
Member

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR migrates the blog plugin’s data import/export wiring from softspring/cms-bundle data components to the newer softspring/cms-data-plugin, and makes the Data services only load when the data plugin is present.

Changes:

  • Conditionally load Data-related service definitions when SfsCmsDataPlugin is available.
  • Update ArticleEntityTransformer to extend/use softspring/cms-data-plugin transformer and support classes.
  • Split Data service registration into a separate data_plugin_services.yaml.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/DependencyInjection/SfsCmsBlogExtension.php Conditionally loads data_plugin_services.yaml when the data plugin class exists.
src/Data/EntityTransformer/ArticleEntityTransformer.php Switches imports/parent class to softspring/cms-data-plugin equivalents.
config/services/services.yaml Removes unconditional registration of Softspring\CmsBlogPlugin\Data\ services.
config/services/data_plugin_services.yaml Adds Data service registration behind the conditional load.
composer.json Adds softspring/cms-data-plugin as a dev dependency.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread composer.json
@javihgil
javihgil merged commit b087f90 into 6.0 Mar 26, 2026
0 of 2 checks passed
@javihgil
javihgil deleted the migrate-import-export-features-to-new-cms-data-plugin branch March 26, 2026 12:55
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