opened
kyleknap
username_0
cc @username_1 @JordonPhillips
Implement copy method for transfer manager
issue
created
jamesls
username_1
Looks good, only question I had was about the str or dict version of the source param.
comment
created
kyleknap
username_0
Should be good to look at again. After our discussion, I just completely removed the ability to pass a string as a copy source as that is a leaky abstraction that should not be used.
comment
created
jamesls
username_1
Looks good, I think it's good that we only allow a dict for the copy source.

Small question about the tests but otherwise looks good.
comment
created
kyleknap
username_0
@username_1 I updated all cases where a shared property was returning a mutable object by prefixing them with ``create_`` to hopefully make that more clear. Let me know what you think.
comment
created
jamesls
username_1
:shipit:  Looks good.
comment