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.

CommentFileSizeAuthor
#1 screen shot41.83 KBmototribe

Comments

mototribe’s picture

mototribe commented
StatusFileSize
new screen shot41.83 KB

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

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
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.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
Status: Closed (won't fix) » Closed (works as designed)