@mikelutz @heddn @quietone @phenaproxima (he/him) @Gábor Hojtsy (he/him) @webchick @Nick Dickinson Wilde @damienmckenna @benjifisher @alison @Seth Green @Joshua Turton (srjosh) @wimleers (he/him) @gabesullice Migration meeting time!! (edited)
0️⃣ Anybody here this week??
benjifisher | Just us :chicken:s. :wink:Benji Fisher, occasional contributor. I am reviewing one issue today. |
heddn | i'm here |
dinarcon | Hello :wave: Mauricio here. |
alison | next week! i'll be there next week!! thanks for being here, y'all! |
1️⃣ what do we need to discuss?
2️⃣ Migrate d7 menu translation
3️⃣ Migration failed with source plugin exception: upgrade_d6_language_content_taxonomy_vocabulary_settings
4️⃣ Allow logging for non-strings values.
5️⃣ Remove lowercase table naming in NodeMigrateTypeTestTrait::getTableName()
6️⃣ Call to a member function getSetting() on null in Drupal\migrate_drupal\Plugin\migrate\EntityReferenceTranslationDeriver->getDerivativeDefinitions() (edited)
7️⃣ MigrateUpgradeImportBatch does not use source_private_file_path & source_base_path correctly, making it impossible to have public & private files in separate locations
8️⃣ Migrations fail due to missing dependency when dependency is clearly not missing.
9️⃣ Allow appending values to multivalued fields during the migration
🔟 Move memory management from MigrateExecutable to an event subscriber
1️⃣ :one: Migrate UI - use human-friendly module names on the Review form
mikelutz | #3024682: Migrate UI - add human-friendly module names to the Review form |
benjifisher | I am reviewing this now. I may not finish by the end of the meeting, but I should finish by EOD. |
benjifisher | @quietone I reviewed the latest patch and requested a few more changes. |
quietone | @benjifisher OK, thanks for the review. New patch posted and passing tests. |
benjifisher | @quietone I knew I could count on you for a quick update! I just finished a re-review: it needs a little more work. |
benjifisher | It looks like a random test failure. I queued a re-test and set the status to RTBC.Let's wait for the test to finish before celebrating. |
benjifisher | @quietone RTBC :tada: |
quietone | Thank you. Thank you! |
1️⃣ 2️⃣ Add base class for migrating reference fields
Participants:
benjifisher, heddn, dinarcon, alison, mikelutz, quietone