Add posibility to show all process plugins not only first one.

  field_ap_technologies:
   -
    plugin: skip_on_empty
    method: process
    source: Technologies
   -
    plugin: explode
    delimiter: ','
   -
    plugin: entity_generate
    ignore_case: true
CommentFileSizeAuthor
#4 after.png11.72 KBalexdoma
#4 before.png11.2 KBalexdoma
#2 2932571-display_all_process_info.patch771 bytesalexdoma
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alexdoma created an issue. See original summary.

alexdoma’s picture

alexdoma CreditAttribution: alexdoma as a volunteer and commented
FileSize
2932571-display_all_process_info.patch771 bytes

Patch for this

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Acro Commerce commented
Status: Active » Needs review

Can we get a screenshot of what this looks like? And any tests you can add would be super nice.

alexdoma’s picture

alexdoma CreditAttribution: alexdoma as a volunteer and commented
FileSize
before.png11.2 KB
after.png11.72 KB

Hi, screenshots were added

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Acro Commerce commented
Status: Needs review » Reviewed & tested by the community

That looks much prettier.

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Acro Commerce commented
Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs reroll

Oops, now we need a re-roll.

alexdoma’s picture

alexdoma CreditAttribution: alexdoma as a volunteer and at Drupal Teams commented

Hi, @heddn !
Can you explain what i should change in my patch? Thank you

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC commented

The changes don't apply on HEAD because of other changes. The instructions for a re-roll: https://www.drupal.org/patch/reroll

alexdoma’s picture

alexdoma CreditAttribution: alexdoma as a volunteer and at Drupal Teams commented

@heddn
I could apply the patch, so no reroll is needed.
i have used last code from here https://git.drupalcode.org/project/migrate_tools

alexdoma’s picture

alexdoma CreditAttribution: alexdoma as a volunteer and at Drupal Teams commented
Status: Needs work » Needs review
heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC commented
Category: Feature request » Task
Status: Needs review » Reviewed & tested by the community
Issue tags: -Needs reroll

Sure enough, it does.

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC commented
Status: Reviewed & tested by the community » Fixed

  • heddn committed c3de8be on 8.x-4.x authored by alexdoma 
    Issue #2932571 by alexdoma, heddn: Display process information of a...

Status: Fixed » Closed (fixed)

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