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. 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

Next video meeting 2023-10-12

0️⃣ Who is here today?

Stephanie lurking
Dan Davis Here
benjifisher Hello!

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.

Dan Davis I am still looking for feedback from @benjifisher on my first attempt to document meeting minutes on a d.o issue.https://drupal.slack.com/archives/C226VLXBP/p1693490743938379
mikelutz (he/him) Looks fine, we just need one of us maintainers to add credit.
benjifisher @dinarcon mentioned this on the issue for the 2023-08-31 meeting: https://www.drupal.org/project/drupal/issues/2797505
mikelutz (he/him) saw that, I assumed we talked about it 2 weeks ago.
Stephanie not sure what next steps are for this? - https://www.drupal.org/project/migrate_plus/issues/3007709
quietone HI. catching up.

2️⃣ Action items. To be added later.

3️⃣ Statistics

mikelutz (he/him) 3 fixed in the last 2 weeks, 8 RTBC, 2 NR both over a month old.
quietone Should be seeing improvements now that committers have returned from holidays

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.

mikelutz (he/him) https://www.drupal.org/project/drupal/issues/3381772
benjifisher I fixed some formatting in the transcript from 2023-08-31:
  • Add
    <code>

    around some code snippets and a few in-line elements.

  • Add some paragraph breaks when a single Slack message had more than one paragraph.
  • I fixed a link to a particular comment on an issue. The d.o text format handles tokens like [12345-67] for Comment #67 on Issue #12345.

These are all things that the Chrome plugin does not handle well.

benjifisher I added credit: include @Dan Davis (ddavisboxleitner on d.o) for adding the transcript and exclude @alison (whose only comments were on the "who is here today" and "wrap up" threads). (edited)
benjifisher When I do the transcript, I normally include even the threads with no comments, like 2️⃣ from 2023-08-31.
Dan Davis Do you add them manually? When using the chrome extension, it’s not clear how I would open an empty thread to copy its contents.
benjifisher Hover over the Slack comment. There are contextual links, including one to open a thread for the comment. The UI is a little unclear, and you have to make sure that you are opening a thread for the correct comment, not off by one.
Dan Davis @benjifisher if you are not already working on the meeting notes for 2023-08-03 I can work on that. https://www.drupal.org/project/drupal/issues/3376391
benjifisher Please do!

6️⃣ Announcements

7️⃣ Migrations fail due to missing dependency when dependency has skipped rows by the source plugin

mikelutz (he/him) https://www.drupal.org/project/drupal/issues/2797505
quietone I haven't look at this in ages.
quietone It just made it a habit to skip rows in the process pipeline.

8️⃣ Add XPath-style filtering ability in JSON data parser plugin

mikelutz (he/him) https://www.drupal.org/project/migrate_plus/issues/3007709
Stephanie @benjifisher I thought I'd set this to "needs review" but I guess I hadn't. Did that now. I think all but one of the requested changes are good, but I left a comment about the nested data. I don't know how to deal with that either.

Comments

benjifisher created an issue. See original summary.

ddavisboxleitner’s picture

Issue summary: View changes
ddavisboxleitner’s picture

Issue summary: View changes
ddavisboxleitner’s picture

Issue summary: View changes
ddavisboxleitner’s picture

Issue summary: View changes

benjifisher’s picture

Issue summary: View changes
Status: Active » Needs review

I fixed formatting and added issue credits. Someone else should check the credits, so I am leaving this issue at NR.

smustgrave’s picture

Status: Needs review » Fixed

Went into slack and #migration channel to 9/14/2023 and all the conversations threads have been captured. And checked the credit seems those who contributed to conversation were credited.

Status: Fixed » Closed (fixed)

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