Core migration issues
Next video meeting 2024-10-10
Hello all, it’s time for the biweekly 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 second 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. #3456078: [meeting] Migrate Meeting 2024-07-18 2100Z)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 👤 emoji. To take a comment or thread off the record, start with a 🚫 emoji.
0️⃣ Who is here today?
| benjifisher | Hello from Barcelona! |
| mikelutz (he/him) | Hello all |
| Dan Davis | Greetings! |
| quietone | Catching up |
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.
| benjifisher | Implement Entity::fields() for migration destinations |
| benjifisher | Field values are not removed if the migration destination contains a slash |
| benjifisher | Convert MigrateSource plugin discovery to attributes |
| benjifisher | Migrations: Mapping tables not updated when entities are deleted |
2️⃣ Action items. To be added later.
| benjifisher | https://www.drupal.org/project/drupal/issues/2630732: review latest MR. @benjifisher |
| benjifisher | https://www.drupal.org/project/drupal/issues/3110619: NR |
| benjifisher | https://www.drupal.org/project/drupal/issues/3421014: Update issue summary. @godotislate |
| benjifisher | https://www.drupal.org/project/drupal/issues/3475576: Close (works as designed) with a comment. @mikelutz (he/him) |
3️⃣ Statistics
| Migrate Initiative Meeting | 351 total, 0 Fixed, 2 RTBC (1 Major), 4 NR |
| Migrate Initiative Meeting | Google sheet for recording stats: https://docs.google.com/spreadsheets/d/1o0Rjlc1vnnLP5bM5P-SMMyGzqn7258hi... |
| benjifisher | I have to update the spreadsheet ^. |
| benjifisher | Done. |
4️⃣ Comment in this thread if you are looking for ways to contribute. Give us some idea of what you would like to do: documentation, code review, testing, project management, ...
5️⃣ Previous minutes.
| Migrate Initiative Meeting | https://www.drupal.org/project/drupal/issues/3471002 (edited) |
| benjifisher | Already Fixed. |
6️⃣ Announcements
| benjifisher | The next meeting (2024-10-10) will be the quarterly video meeting. |
7️⃣ Implement Entity::fields() for migration destinations
| Migrate Initiative Meeting | https://www.drupal.org/project/drupal/issues/2630732 (edited) |
| Migrate Initiative Meeting | (Original Request) https://drupal.slack.com/archives/C226VLXBP/p1727360126543769?thread_ts=... (edited) |
| benjifisher | I have already reviewed this issue a few times. I will review the latest changes to the MR, maybe tomorrow (contribution day at DrupalCon). |
| benjifisher | I reviewed the issue. I just found one line that still needs to be updated. Cc @joachim |
| joachim | Are you at Barcelona? I’m on the train home. Sorry I didn’t get to say hello. |
| benjifisher | Yes, I am. I met a lot of people for the first time. I am sorry I missed you! |
| benjifisher | RTBC. :+1::status_rtbc: |
| joachim | Wahey!!! Thanks! |
8️⃣ Field values are not removed if the migration destination contains a slash
| Migrate Initiative Meeting | https://www.drupal.org/project/drupal/issues/3110619 (edited) |
| Migrate Initiative Meeting | (Original Request) https://drupal.slack.com/archives/C226VLXBP/p1727360223002459?thread_ts=... (edited) |
9️⃣ Convert MigrateSource plugin discovery to attributes
| Migrate Initiative Meeting | https://www.drupal.org/project/drupal/issues/3421014 (edited) |
| Migrate Initiative Meeting | (Original Request) https://drupal.slack.com/archives/C226VLXBP/p1727360288201249?thread_ts=... (edited) |
| benjifisher | There are 3 MRs:6780: +1363/-7158070: +1453/-6309571: +245/-92 |
| benjifisher | I do not have time to dig in right now. I asked on the issue for an update to the description because the "canonical MR" is 6780, but that is marked as Draft. |
| godotislate | I can update the IS later, but https://git.drupalcode.org/project/drupal/-/merge_requests/6780 isn't draft |
| godotislate | it is however stalled bc we need to deal with a Trait |
| godotislate | details are in my latest comments |
| benjifisher | Oops, 8070 is the one that is marked Draft. |
| godotislate | IS update will come later bc it will be a big rewrite |
| godotislate | the issue is a mess |
| benjifisher | Sorry to hear that. |
| benjifisher | Should it be marked NW? |
| godotislate | it's in NR for feedback on approach, but yeah, can put it back until the IS update |
| godotislate | ok, moved the ticket to NW |
| quietone | Thanks for working on this. This is a child of a priority issue noted so on[#3462359] |
| godotislate | npfinally updated the IS and rebased MR |
🔟 Migrations: Mapping tables not updated when entities are deleted
| Migrate Initiative Meeting | https://www.drupal.org/project/drupal/issues/3475576 (edited) |
| Migrate Initiative Meeting | (Original Request) https://drupal.slack.com/archives/C226VLXBP/p1727360347753699?thread_ts=... (edited) |
| benjifisher | @quietone already suggested closing this: works as designed. I think I agree. |
| mikelutz (he/him) | Done with a comment. |
| benjifisher | Thanks! |
| quietone | Yes, thanks! |
1️⃣1️⃣ Wrap-Up
| Migrate Initiative Meeting | Thanks for coming all! See you in 2 weeks |
Participants:
benjifisher, Migrate Initiative Meeting, joachim, godotislate, quietone, mikelutz
Comments
Comment #2
ddavisboxleitner commentedComment #3
benjifisherComment #8
smustgrave commentedWent into the #migration channel and appears all threads captured. Added the credits.