search

Jumoo / uSyncMigrations

Rough and ready migration code.
Mozilla Public License 2.0
44 stars 58 forks source link

Ability to rename content types #209

Closed michaelchart closed 1 year ago

michaelchart commented 1 year ago

Based on comments from #200.

I've taken over the work that @bielu started. We found some issues with his implementation - where the replacement aliases weren't being used in all places. This PR fixes those issues and addresses the feedback from #200.

I tried to cover any other places that content type aliases are referenced such as in MNTP config, but it's possible there are others I've missed. Hopefully this is a good enough starting point.

KevinJump commented 1 year ago

Hi,

So just to confirm this superseeds #200 - (which i haven't merged yet)

michaelchart commented 1 year ago

@KevinJump Yes, that's right

KevinJump commented 1 year ago

Cool

KevinJump commented 1 year ago

Hi,

I've added a really small change, in that I've put in an obsoleted GetAliasAndKey method, (meaning it will call a null context) - it will just help anyone who might have with a custom implementation as it will at least work and warn them.

(we are still in preview, so yeah, we can just break, but sometime i like to do it a little slowly 😄 )