Problem/Motivation

1. The original report (see below) pointed out that documentation, change record, and the code base were not in agreement about the location of the migration yml files.

The change record is correct. Migration yml files should be located in MODULE/migrations instead of MODULE/migration_templates.

2.Requires Configuration Tag to migration to specify configuration migration.

Steps to reproduce

Proposed resolution

1.Rename all directories from contrib/module_name/migration_templates to contrib/module_name/migrations.

2.Add Configurationas migration_tags.

Remaining tasks

N/A

User interface changes

N/A

API changes

N/A

Data model changes

N/A

CommentFileSizeAuthor
#4 simplesamlphp_auth-update_migration-3256410-3.patch15.15 KBomkar.podey
#2 email_registration-update_migration-3256294-2.patch588 bytesomkar.podey

Comments

omkar.podey created an issue. See original summary.

omkar.podey’s picture

omkar.podey commented
StatusFileSize
new email_registration-update_migration-3256294-2.patch588 bytes

Changed migration file location to migrations folder , updated migration tags.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

Thanks for your contribution.

Could you add some details using the template to make the issue summary a little more complete? That would help folks understand what's going on here and why it's important.

omkar.podey’s picture

omkar.podey commented
StatusFileSize
new simplesamlphp_auth-update_migration-3256410-3.patch15.15 KB

wrong patch don't mind

omkar.podey’s picture

omkar.podey commented
omkar.podey’s picture

omkar.podey commented
omkar.podey’s picture

omkar.podey commented
Issue summary: View changes
omkar.podey’s picture

omkar.podey commented
Status: Active » Needs review
narendrar’s picture

narendrar
He/Him
Primary language English
Location Jaipur, India
commented
Issue summary: View changes
narendrar’s picture

narendrar
He/Him
Primary language English
Location Jaipur, India
commented
Status: Needs review » Reviewed & tested by the community

Tested manually and migration is working from D7 to D9.

  • greggles committed df0d5e2 on 8.x-1.x authored by omkar.podey 
    Issue #3256294 by omkar.podey, narendraR, greggles: Updated Migration...
greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Reviewed & tested by the community » Fixed

thanks for the patch and review. Now committed.

Status: Fixed » Closed (fixed)

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