Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 292776

[meeting] Migrate Meeting 2020-04-02

$
0
0

@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??

benjifisherJust us :chicken:s. :wink:Benji Fisher, occasional contributor. I am reviewing one issue today.
heddni'm here
dinarconHello :wave: Mauricio here.
alisonnext 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

mikelutz#3112249: Migrate d7 menu translation

3️⃣ Migration failed with source plugin exception: upgrade_d6_language_content_taxonomy_vocabulary_settings

mikelutz#3101045: LanguageContentSettingsTaxonomyVocabulary source plugin should only add language column if it exists

4️⃣ Allow logging for non-strings values.

mikelutz#3047328: Allow logging for non-strings values

5️⃣ Remove lowercase table naming in NodeMigrateTypeTestTrait::getTableName()

mikelutz#3119710: 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) 

mikelutz#2984460: Call to a member function getSetting() on null in Drupal\migrate_drupal\Plugin\migrate\EntityReferenceTranslationDeriver->getDerivativeDefinitions()

7️⃣ MigrateUpgradeImportBatch does not use source_private_file_path & source_base_path correctly, making it impossible to have public & private files in separate locations

mikelutz#2925899: 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.

mikelutz#2797505: Migrations fail due to missing dependency when dependency is clearly not missing

9️⃣ Allow appending values to multivalued fields during the migration

mikelutz#2992168: Allow appending values to multivalued fields during the migration

🔟 Move memory management from MigrateExecutable to an event subscriber

mikelutz#3006750: Move memory management from MigrateExecutable to an event subscriber
heddn@mikelutz this could use a review for BC concerns at this point. and I wrote too much of it to be too unbiased

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
benjifisherI 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.
benjifisherIt 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:
quietoneThank you. Thank you!

1️⃣ 2️⃣ Add base class for migrating reference fields

heddn#2447727: Add base class for migrating reference fields
heddnReviewing

Participants:

benjifisher, heddn, dinarcon, alison, mikelutz, quietone


Viewing all articles
Browse latest Browse all 292776

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>