Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 297447

[meeting] Migrate Meeting 2020-02-20

$
0
0

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: https://www.drupal.org/project/drupal/issues/3113541
➤*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.

0️⃣ Who is here today? Introduce yourself in this thread.

benjifisherHello! I have not been doing any migration work for the last few months, but I have some free time now, so I will see if there are any issues where I can help. Does anything need usability review?
Gábor Hojtsy (he/him)Hi!
heddnhowdy
wimleers (he/him):wave:
damienmckennaStill Damien.
hashHello Friends, I have done migration on Drupal 8 with source as XML and previously used migrate module in Drupal 7, I am looking to see if I can help the community in fixing any issues.
Joshua Turton (srjosh)hello!

1️⃣ What do we need to talk about?

Gábor Hojtsy (he/him)Future of migrate in Drupal 9/10?
Joshua Turton (srjosh)Is it even possible to migrate multilingual entity references?
mikelutzIt is.

2️⃣ Migration translation revisions

mikelutz#2746541: Migrate D6 and D7 node revision translations to D8
mikelutzI’ve started reviewing this one, there are a few outstanding questions about some fixture and test changes I made way back in my first proof of concept patch. I know I had reasons at the time, but I’m looking through before and after fixtures right now to try to remember specifics.
Gábor Hojtsy (he/him)It would be really nice to see this land, as @quietone would be able to continue working on the entity translation one which is also needed to land before we can stabilise the migrate multilingual stuff
Gábor Hojtsy (he/him)does @heddn have interest/time for this (too)? :slightly_smiling_face:
heddnmy biggest priority when I get some free moments is #3093652: Make migrate_upgrade compatible with Drush 10.. which is also a pretty big blocker for things too.
heddn@benjifisher not UX, but possibly an area you could help is to get tests passing on ^. It would free up some time.
benjifisherI will have a look. How hard can it be? (famous last words)

3️⃣ getHighestId() should be able to use any integer destination id

mikelutz#3091004: getHighestId() should be able to use any integer destination id
mikelutzThis has been in Needs Review for over 3 months.
mikelutzI’m setting it back to NW
benjifisherIt looks like a follow-up to an issue I reviewed. I can review it before next week's meeting.

4️⃣ Implement Entity::fields() for migration destinations

mikelutz#2630732: Implement Entity::fields() for migration destinations
mikelutzThis one has been in NR for 3 weeks.
mikelutzI set this to NW as well

5️⃣ Create a way to declare a plugin ID defined in a migration yml file as deprecated

mikelutz#3039240: Create a way to declare a plugin ID defined in a yml file as deprecated
mikelutzI set this to postponed for the moment, though I expect people to disagree.

6️⃣ Plugin migration_lookup don’t process source when part of plugin sub_process

mikelutz#2915717: Plugin migration_lookup don't process source when part of plugin sub_process
mikelutzClosed WaD

7️⃣ Migrate d7 menu translation

mikelutz#3112249: Migrate d7 menu translation

8️⃣ Optimize migration of specific source IDs for SQL sources

mikelutz#2780839: Optimize migration of specific source IDs for SQL sources

9️⃣ Move actions migrations and tests to system module

mikelutz#3067299: Move actions migrations and tests to system module

🔟 Migrate “findMigrationDependencies” function throws error when “process” param is null (edited) 

mikelutz#2981837: Migrate "findMigrationDependencies" function throws error when "process" param is null
mikelutzClosing WaD

1️⃣ :one: Migration of menu links does not set the appropriate bundle

mikelutz#3070495: Migration of menu links does not set the appropriate bundle

1️⃣ 2️⃣ Fix “MigrateCckField is deprecated in Drupal 8.3.x and will be removed before Drupal 9.0.x. Use \Drupal\migrate_drupal\Annotation\MigrateField instead.” deprecation error

mikelutz#2970108: Fix "MigrateCckField is deprecated in Drupal 8.3.x and will be removed before Drupal 9.0.x. Use \Drupal\migrate_drupal\Annotation\MigrateField instead." deprecation error

1️⃣ 3️⃣ If no Drupal 8 equivalent filter is found during migration, drop transformation-only filters, so that body fields show up

mikelutz#3061571: If no Drupal 8 equivalent filter is found during migration, drop transformation-only filters, so that body fields show up
wimleers (he/him):pray:
mikelutzRTBC

1️⃣ 4️⃣ Field formatter & widget settings: fall back to default if missing plugin

mikelutz#3108302: Field formatter & widget settings: fall back to default if missing plugin

1️⃣ 5️⃣ Migrate empty, and link field

mikelutz#3110064: Migrate empty, <nolink> and <none> link field
hashcan I assign this to myself, seems doable as I understand how I can reproduce and test it ? @mikelutz
mikelutzhttps://www.drupal.org/patch/review
hashthanks @mikelutz

1️⃣ 6️⃣ Future of Migrate in Drupal 10 and beyond

mikelutzThere has been some discussion about what the future of migrate is once Drupal 7 marketshare is sufficiently reduced and it no longer makes sense to maintain the Drupal 6/7 migrations in core.
mikelutzI don’t actually believe that d7 marketshare will be low enough by Drupal 10 to remove the migrate_drupal module, but it will happen eventually. One of the thresholds we’ve used as to whether a feature goes in contrib or in core is whether it directly supports the core mission of providing Drupal 6/7 migrations into drupal 8, so what happens once that system is nol longer needed.
benjifisherI saw a suggestion from Moshe that the Migrate API move back to contrib in D10. I guess that means he is volunteering. :wink:
mikelutzThe migration system has proved itself to be very useful in other ways, and I belive it has value in core as an api. It’s useful for pulling in data from all kinds of sources (though it could continue to do so in contrib)
heddnwell, in d10, it might make sense to use something like soong instead. too early to tell
mikelutzI believe the best business case for keeping it in core would be as a system focused on pulling in data from other competing CMS systems.
heddnhaving the destination plugins are pretty useful in core. and the base process and migration plugins. the source plugins for things besides the base sql could probably get deprecated
mikelutzThe biggest selling point of migrate is that it lowers the barrier to entry to Drupal when coming from some other system. Given the way our market share looks, I don’t think we can give that up or send it back to contrib.
mikelutzBut I do think it really changes what migrate looks like in the coming years.
mikelutzFocusing on improving the robustness and usefulness of the API outside of just what core happens to currently use for d6/7 migrations (whether that is soong or not)
benjifisherIs this a good summary of your suggestions?Keep the Migrate API (migrate module) in core.Consider removing migrate_drupal and migrate_drupal_ui modules to contrib when D7 market share drops "enough".
mikelutzWell, also at some point to consider bringing more of migrate plus and tools into core, regardless of whether core migrations use things. I think to do that will require us to define a new threshold for what’s allowed, otherwise we risk a bloat of very specialized process plugins, among other things.
benjifisherRight. The current criterion is to include things needed by migrate_drupal(). If we remove that module, then we need a new criterion.
benjifisherHere is a half-baked thought: the new criterion could be to support migrations from JSON:API (the actual API, not the module that implements it in Drupal).
mikelutzNot a bad thought, but migrate doesn’t do well with that type of a source at all right now.
mikelutzWith that being a top down get the parent, then find the references then the references of references while migrate likes to process the references first and associate them with a parent afterward.
andypost5c is that feeds making nice UI on top of migrate, so +1 to UI in contrib
kevinquillenBetter support for JSON/XML parsing and updating from, to handle ongoing migrations
kevinquillen(similar to feeds but for migrate)
andypost#2990289: [META] feeds_migrate Road Map
kevinquillenIs this Feeds or Migrate though? I want to stick with MIgrate, and have no need of UI for it

Participants:

benjifisher, Gábor Hojtsy (he/him), heddn, wimleers (he/him), damienmckenna, hash, Joshua Turton (srjosh), mikelutz, andypost, kevinquillen


Viewing all articles
Browse latest Browse all 297447

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>