- Issue created by @himanshu_jhaloya
- @himanshu_jhaloya opened merge request.
- 🇨🇦Canada leevh
I might be wrong, but this functionality seems crucial when the tid is what links terms to content. Otherwise the tid seems to keep changing when using structure sync. Thanks for this!
- Status changed to Needs review
7 months ago 12:44pm 9 October 2024 - 🇫🇷France simon georges Rouen, France
This is an updated version of the patch for the latest version of the module.
- First commit to issue fork.
- 🇨🇦Canada mparker17 UTC-4
To make this easier to review, I've rebased the merge request onto the latest 2.x.
A brief glance at the code in the merge request suggests that it has no tests. The Structure Sync maintainers prefer to accept merge requests that have passing automated tests. If you need help writing tests, please ask: I would be happy to help!
I've marked this issue as "Needs title update" and "Needs issue summary update" because it's unclear to me what feature you are trying to add to the module nor what feature is being fixed. I took a look at the code too, but the intent behind the change is still unclear to me.