Jumoo / uSyncMigrations

Rough and ready migration code.
Mozilla Public License 2.0
48 stars 62 forks source link

Idea: Community: Added migrator for StackedContent to BlockList #41

Closed leekelleher closed 1 year ago

leekelleher commented 1 year ago

Add a community migrator for Stacked Content to latest Block List editor.

References for data structure:

I'd considered a migrator to Contentment Content Blocks, but felt that would muddy the waters for default/standard uSync migrations.

soreng commented 1 year ago

hi @leekelleher

I can't seem to figure out how to add to your branch, but I have opened another PR with the tweaks I was able to implement and test today.

One hurdle I walked into was that the solution I was testing a migration from, is using nested content inside stacked content, resulting in a error in umbraco. This is by design, but took me a bit to figure out.

leekelleher commented 1 year ago

I can't seem to figure out how to add to your branch, but I have opened another PR with the tweaks I was able to implement and test today.

@soreng The way to do it is slightly long winded. You'd typically need to submit the PR on my forked repo, e.g. https://github.com/leekelleher/umbraco-usync-migrations, targeting the feature/stacked-content-to-block-list branch. Then I'd merge that in, and it'd update PR #44 on this repo.

But I appreciate that is long winded, so going with PR #57 is fine. It includes my original commit, so I'm happy that I'll still be credited with the initial code - all good with me. 👍