Right now, at the completion of migration any messages shown are displayed as green/success, even if there were failures in some migrations. We need to use appropriate statuses.

CommentFileSizeAuthor
#3 migrate-status-message-2282041-3.patch2 KBer.pushpinderrana
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

er.pushpinderrana’s picture

er.pushpinderrana CreditAttribution: er.pushpinderrana commented
Version: » 8.x-1.x-dev
Assigned: Unassigned » er.pushpinderrana
Status: Active » Needs work

I reproduced this issue at my end, will submit patch soon.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan commented
Status: Needs work » Active

The "Needs work" status would be if there was a submitted patch already that, well, needs work.

Thanks for looking at this.

er.pushpinderrana’s picture

er.pushpinderrana CreditAttribution: er.pushpinderrana commented
Assigned: er.pushpinderrana » Unassigned
Status: Active » Needs review
FileSize
migrate-status-message-2282041-3.patch2 KB

Please review attached patch once.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan commented
Status: Needs review » Fixed

Committed (minus a gratuitous !), thanks!

Status: Fixed » Closed (fixed)

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