Conversation
JoshuaLavarine
left a comment
There was a problem hiding this comment.
This looks good! I have one question, which I've noticed here and also referenced in a thread from my PR in production_transporter.
Do you think that we need to standardize the article_id type within Editorial Manager Transfer Service? There are a few instances of str type remaining, while the majority are int type.
They should all be "str" as that is the proper type. I thought I got them all. |
Updates the plugin to add more realistic testing and log transfer success and failures.