Hello all, it’s time for the weekly migration subsystem 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 on the #migration channel in Drupal Slack (see www.drupal.org/slack for information).
➤ 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. See the parent issue for an idea of the typical agenda.
➤*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.
Core migration issues
Video meeting 2022-10-13 2100Z (tentative)
The hope is that most or all of the maintainers will attend. We will try to focus on longer-term goals than in the weekly meeting.
0️⃣ Who is here today?
| benjifisher | Hello! I am headed to Denver tomorrow for DrupalCamp CO. |
| quietone | Hi, very late. |
1️⃣ What should we talk about today? Suggest topics here and I will add threads. I will also check for comments on the issue for today’s meeting.
| bbombachini | Idk if it’s a good moment for this or not but I just realized that on this page here (https://www.drupal.org/docs/8/api/migrate-api/migrate-process-plugins/li...), all of the process plugins’ links take you to an error page. I think now we need to specify the branch?Ex: https://cgit.drupalcode.org/migrate_plus/tree/src/Plugin/migrate/process... |
2️⃣ Action items. To be added later.
3️⃣ Statistics
| mikelutz (he/him) | NR -19, 3 major |
| mikelutz (he/him) | RTBC - 4, one crit |
| mikelutz (he/him) | fixed since last meeting: 0 |
| mikelutz (he/him) | https://docs.google.com/spreadsheets/d/1o0Rjlc1vnnLP5bM5P-SMMyGzqn7258hi... |
4️⃣ Comment in this thread if you are looking for ways to help. Give us some idea of what you would like to do: documentation, code review, testing, project management, ...
5️⃣ Previous minutes.
| mikelutz (he/him) | #3300474: [meeting] Migrate Meeting 2022-07-28 1400Z |
| benjifisher | I think I can add a transcript. |
| benjifisher | Done. NR |
| benjifisher | Also NR: #3296087: [meeting] Migrate Meeting 2022-07-21 2100Z (video meeting from two weeks ago). |
6️⃣ Announcements
7️⃣ $migration_dependencies has inconsistent structure
| mikelutz (he/him) | #3262395: $migration_dependencies has inconsistent structure |
| benjifisher | Last week, I cleaned up the MR and the test passed.
This issue was previously RTBC. I updated it to target 10.1.x because it has new deprecations. I hope someone can review that update. |
| quietone | I've added this to my list |
8️⃣ Regression AND bug: SQLite source and id map tables aren’t joinable anymore, but SqlBase::mapJoinable() is unaware of this
| mikelutz (he/him) | #3260111: Regression AND bug: SQLite source and id map tables aren't joinable anymore, but SqlBase::mapJoinable() is unaware of this |
| mikelutz (he/him) | Gonna move this to postponed, maintainer needs more info, since there has been no response from the OP or anyone else that I thought had a stake in this. |
| benjifisher | Fair enough. You asked for feedback in your comment on the issue on 2022-06-26.
Last week, we confirmed that the failing test was a sporadic failure. Let's try once more to get some feedback: @wimleers (he/him), @huzooka, do either of you still care about this one? |
9️⃣ Expose full set of debugging data in migrate_message table (filterable/searchable)
| mikelutz (he/him) | #3063856: Add ability to view migrate_message table data |
| mikelutz (he/him) | #3063856: Add ability to view migrate_message table data |
| benjifisher | It is a quiet meeting today, so I will have a look at that. |
| benjifisher | I did a little manual testing and left a comment. So far, :+1: |
| benjifisher | Uh-oh, there is some untranslated text in there. |
| quietone | I've added this to my list |
Comments
Comment #4
quietone commentedComment #5
quietone commentedComment #6
benjifisherI checked against the Slack discussion and confirmed the credits. LGTM