It's broken - fix it.

Comments

mikeryan created an issue. See original summary.

  • mikeryan committed 42524aa on 8.x-2.x
    Issue #2690345 by mikeryan: Fix group configuration merging for 8.1.x
    
mikeryan’s picture

Status: Active » Fixed

Done. Bonus - since we now control the migration configuration entity, we can make the group a proper property rather than resorting to third_party_settings.

agoradesign’s picture

I've followed your changes in the example for my custom migrations. But when I call drush migrate-status, everything gets listed under "default" group. Can you confirm that this works for you?

mikeryan’s picture

Yes, it works for me with migrate_example. Note per my comment above that the migration_group property has been moved out of third_party_settings, you should be setting that as a top-level property in your migration .yml files.

agoradesign’s picture

Yeah, I know.. I've moved them, and I compared my migration files with the beer example.... strange. I will look at this again

agoradesign’s picture

Now it's clear... you were working with a patched migrate_tools already, After you committed http://cgit.drupalcode.org/migrate_tools/commit/?id=1893c98 and update to the latest dev, the groups are working as expected for me too

mikeryan’s picture

Yep, I had neglected to commit and push that change, finally did so...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.