Download & Extend

better display of error messages

Project:Migrate
Version:7.x-2.2-rc1
Component:migrate_ui
Category:feature request
Priority:minor
Assigned:Unassigned
Status:closed (works as designed)

Issue Summary

First of all, thanks a bunch for such a cool module. It takes a while to learn but is very, very powerful.

I just spent an hour trying to figure out why I couldn't import an addressfield. Turns out that I forgot to include the countrycode
in the query, ouch. It showed the message on the screen, I just didn't notice:

Only local images are allowed.

Maybe a red background would be better? Also, why would migrate even run in this example? I would have expected it to not even import.

Comments

#1

sorry, here is the screen shot (error message on yellow background)

AttachmentSize
screen shot 41.83 KB

#2

Status:active» closed (won't fix)

The message really doesn't seem that subtle to me, it's clearly visible. It's appropriate to the condition, which is not a fatal error. In many scenarios the source field list is explicitly built by the implementor in the constructor, and the absence of a field from the source field list is really just a documentation issue, the actual migration will run correctly.

#3

Status:closed (won't fix)» closed (works as designed)
nobody click here