- Issue created by @majorrobot
- Merge request !9Issue #3538461: Create a data parser migration_plus plugin β (Merged) created by majorrobot
- πΊπΈUnited States majorrobot
MR is ready for review.
- Adds an Ai data_parser plugin that utilizes our AiMigrator service.
- Adds a test migration yml for an example slice of urls.
- Adds an additional config file for our sample content type. It is necessary for loading the content type during tests.
- Updates AiMigrator so that convert() takes url, html, or both. Improves Ai prompt.
- Updates the hard-coded bundle structure for AiMigrator so that it resembles an actual Drupal bundle and can be migrated easily.
- Updates tests.
- πΊπΈUnited States dmundra Eugene, OR
Ran into some bugs trying to enable the example module after re-doing my local site.
- πΊπΈUnited States majorrobot
Sry about that @dmundra. There was a extraneous field storage config I had added to address a test that I later removed. I think I've resolved the issue now.
Yes, a good test command is
ddev drush mim slice1
.Thank you!
- πΊπΈUnited States dmundra Eugene, OR
@majorrobot migration working well. Noticed that the example content-type didn't display fields on node view or edit so after enabling those I can see the values. I will share comparing AI response with what is being migrated.
- πΊπΈUnited States dmundra Eugene, OR
For the HTML and other content type issues that can be handled in future issues like <#3539369>
- πΊπΈUnited States dmundra Eugene, OR
We mob-programmed on this PR so credited all the folks. Thank you!
- πΊπΈUnited States majorrobot
Thx to everyone who mobbed and helped us resolve issues.
@dmundra, I've committed all of the suggestions. This is ready to go again.
- πΊπΈUnited States dmundra Eugene, OR
Looking good @majorrobot. Approved and merged.
-
dmundra β
committed 8343043f on 1.0.x authored by
majorrobot β
Issue #3538461: Create a data parser migration_plus plugin
-
dmundra β
committed 8343043f on 1.0.x authored by
majorrobot β