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

benjifisher created an issue. See original summary.

ddavisboxleitner’s picture

Issue summary: View changes
benjifisher’s picture

Status: Active » Needs review

smustgrave’s picture

Status: Needs review » Fixed

Went into the #migration channel and appears all threads captured. Added the credits.

Status: Fixed » Closed (fixed)

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