verbb / hyper

A Craft CMS plugin for managing links, focusing on user experience.
Other
24 stars 15 forks source link

Content Migration does not work in Neo fields in all site groups #116

Closed sword3d closed 8 months ago

sword3d commented 8 months ago

Describe the bug

We have two site groups. Group A has two sites, Group B has 5 sites. Content migration works fine in non-Neo fields. But doesn’t work in Group B in Neo fields. Group A is fine. I’m attaching some screenshot. Happy to provide more debugging information.

Steps to reproduce

Screenshot_1 Screenshot_2 Screenshot_3

Craft CMS version

Craft CMS 4.7.4

Plugin version

1.1.21

Multi-site?

Yes

Additional context

We are migrating from "sebastianlenz/linkfield": "2.1.5"

engram-design commented 8 months ago

So I can confirm things are working correctly for multi-sites, but might be unique to multiple groups.

Can you confirm your Neo field "Propagation Method" and the "Translation Method" of your Typed Link field?

sword3d commented 8 months ago

Typed Link field -> "Translation Method" -> Translate for each site Neo field -> "Propagation Method" -> Save blocks to other sites in the same site group

engram-design commented 8 months ago

Okay, just testing that and the migration is working as expected on my end, with content being migrated across properly.

Any errors in the migration feedback for either the fields themselves, or the content?

sword3d commented 8 months ago

No errors. Screenshot_5

engram-design commented 8 months ago

Thanks for confirming, that does indeed seem fine. Any chance of being able to send your database from before your migration to Hyper? If so, send to support@verbb.io and we can get to the bottom of it. Otherwise, we'll keep at replicating it.

sword3d commented 8 months ago

Sent it by mail.

engram-design commented 8 months ago

Thanks for sending that through, that was a big help. I'm not sure if my link field is out of date, or yours is, but looks like we weren't handling the siteId of linked elements correctly for Typed Link migration. This is despite it actually working on my end, but I could replicate it with your setup.

Fixed for the next release. To get this early run composer require verbb/hyper:"dev-craft-4 as 1.1.22"

sword3d commented 8 months ago

Thank you for the fix. I can confirm that this works.

engram-design commented 8 months ago

Fixed in 1.1.23