- 🇹🇷Turkey alexander.levitsky Antalya
alexander.levitsky → made their first commit to this issue’s fork.
- @alexanderlevitsky opened merge request.
- 🇹🇷Turkey alexander.levitsky Antalya
Hi fisherman90,
Really thank you for your solution, but let's keep the only drush command in the scope of this issue, without any code style fixes, because it conflicts with other patches.
I have prepared another PR with your drush command only.
Btw, it works perfectly for me as well.
- 🇨🇦Canada liquidcms
I guess since i dont see UUIDs in the export yml this cant be used to sync existing menus, only to create new ones?
also, a bit unclear the what this adds with these new commands. The README already stated that the std CEX/CIM works for this. I think CEX sort of works and CIM doesn't.
I tried the menu_export:import and i get:
SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '231' for key 'menu_link_content__revisio
n_id'i assume as my menu already exists and i am just trying to update it.
- 🇩🇰Denmark ressa Copenhagen
I can't see why a Drush export command is needed? I just installed the module, added a menu, ran
drush config:export, and the menu was exported in a yml file, as expected. I did not need to click the "export" button.So I just posted this comment 📌 Step by Step Instructions Here (I Think) Active , assuming that since
drush config:exportincludes the yml-files without any further steps, I assumed that "drush config:import" would import the menu links ...So do you need to manually click "import", but don't need to manually click "export"?
- Status changed to Needs work
over 1 year ago 11:22am 29 October 2023 - 🇩🇰Denmark ressa Copenhagen
It does look like the menus are not imported, and you actively need to click the "Import menu links" button. The patch needs a re-roll, so changing Status.
- First commit to issue fork.
- Status changed to Needs review
12 months ago 7:22pm 14 July 2024 - 🇺🇸United States mortona2k Seattle
I rebased the branch on 1.x.
Had to change the services file to drush.services.yml to get it to recognize the command.
To fix the errors in #10, apply the patch from here: https://www.drupal.org/project/menu_export/issues/3399829#comment-15484598 🐛 Menu Import broken in Drupal 10 Needs review
- 🇩🇰Denmark ressa Copenhagen
Until this gets committed, https://www.drupal.org/project/menu_migration → has Drush support, and even translatable menus.