https://drupal.slack.com/archives/C226VLXBP/p1588886146256900

Hello all, it’s time for the weekly migration initiative meeting. The meeting will take place in slack in various threads
This meeting:
➤ Is for core migrate maintainers and developers and anybody else in the community with an interest in migrations
➤ Usually happens every Thursday and alternates between 1400 and 2100 UTC.
➤ Is done over chat.
➤ Happens in threads, which you can follow to be notified of new replies even if you don’t comment in the thread. You may also join the meeting later and participate asynchronously!
➤ Has a public agenda anyone can add to here:
➤*Transcript will be exported and posted* to the agenda issue. For anonymous comments, start with a :bust_in_silhouette: emoji. To take a comment or thread off the record, start with a :no_entry_sign: emoji.

1️⃣ Who is here?

mikelutz(opens in new tab) not me (hides)
benjifisher(opens in new tab) Benji Fisher, occasional contributor. I often review issues, but do not expect a rubber-stamp RTBC. :wink:
dinarcon(opens in new tab) Mauricio :wave:

2️⃣ What topics should we discuss?

benjifisher(opens in new tab) We got this one back to RTBC: #3024682: Migrate UI - add human-friendly module names to the Review form :tada::balloon: :boom:
benjifisher(opens in new tab) #3096101: Allow migrate_drupal_ui source database to be set in settings.php I see it is tagged for Usability review, so I will take it up at next week's Usability meeting.
quietone(opens in new tab) Thanks.
quietone(opens in new tab) On the positive side we have 6 RTBC issues.

3️⃣ Automatically derive auxiliary migrations. #2746517: Automatically derive auxiliary migrations

quietone(opens in new tab) I am hoping we can close this.

4️⃣ Rollback of complete node migration fails. #3123095: Rollback of complete node migration fails

quietone(opens in new tab) Anyone working with the complete node migration needs this, hard to develop without it.
benjifisher(opens in new tab) This is the only one that is marked Major.
benjifisher(opens in new tab) The patch is pretty small. I can make this my goal for the next week.@quietone is the failing patch in #2 unchanged in the later patches? If not, then I would like to have a new test-only patch.
quietone(opens in new tab) I think it is but I will double check and leave a comment today.

5️⃣ Switch to entity owner in EntityContentBase during validation #3134470: Switch to entity owner in EntityContentBase during validation

6️⃣ Memory management #3006750: Remove memory management from MigrateExecutable

I could go one but we seem pretty quiet. Anything in particular on your mind?

dinarcon(opens in new tab) Do we need new sections on the documentation for D9 specific content? I guess it will be mostly the same, at least at the beginning.

7️⃣ Allow migrate_drupal_ui source database to be set in settings.php

benjifisher(opens in new tab) [#3096101]I just set it to NW: it needs an issue summary update before it can get a usability review.
quietone(opens in new tab) Thanks. I see what I can do

8️⃣ correct drupal6 test fixture

quietone(opens in new tab) #3116841: Correct latest revision for node 1 in drupal6 test fixture
quietone(opens in new tab) @mikelutz Do we need to do more work here about the earlier comment failure?

9️⃣ Documentation: List of core Migrate process plugins

benjifisher(opens in new tab) https://www.drupal.org/docs/8/api/migrate-api/migrate-process-plugins/li...
benjifisher(opens in new tab) This page lists the process plugins in the core migrate module and has a link to the page of process plugins in the contrib Migrate Plus module.
dinarcon(opens in new tab) I wonder if the link should be updated to something like /docs/8-9 or a new section /docs/9 should be created. At, api.drupal.org there are tabs to switch among versions. Is that something we want to do? I guess this goes beyond migrate documentation. Many subsystems will require something similar.
benjifisher(opens in new tab) If we want to be consistent with the title, I think we shouldMove the link from the body of the page to the Related content section (currently empty).While we are at it, do the same for the link to "Writing a custom process plugin".Add links to the API docs of process plugins provided by other core modules.
benjifisher(opens in new tab) @dinarcon I think that is outside the scope of this Migrate API meeting. I am pretty sure it is already under discussion.
dinarcon(opens in new tab) Got it. I have been referring to the docs a lot lately and was wondering about that. But yes, it is out of scope.

1️⃣ 0️⃣ Thanks and we hope to "see" you next week. (edited) 

benjifisher(opens in new tab) Please continue to add comments to the threads above. I volunteer to copy the discussion to an issue some time tomorrow unless someone else wants to do it.
quietone(opens in new tab) Thank you!!
alison(opens in new tab) Thank you for running, @benjifisher + @quietone !!!!!!! (edited)

Participants:

mikelutz(opens in new tab), benjifisher(opens in new tab), dinarcon(opens in new tab), quietone(opens in new tab), alison(opens in new tab)

Comments

benjifisher created an issue. See original summary.

benjifisher’s picture

benjifisher’s picture

Status: Active » Fixed

Status: Fixed » Closed (fixed)

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