search

astrikos / wetransfer-python-sdk

:snake: A Python SDK for WeTransfer's Public API
GNU General Public License v3.0
3 stars 1 forks source link

Bug in Transfer.add_links? #3

Closed fservantwiremind closed 6 years ago

fservantwiremind commented 6 years ago

Hello, Using wetransfer-0.0.1, in the implementation of Transfer.add_links it is written:

self.transfer_items.append(File(url))

I guess you would have wanted to write

self.transfer_items.append(Link(url))

instead?

astrikos commented 6 years ago

🤦‍♂️ you are right ofcourse! The lack of tests has already started to become obvious. Feel free to submit a PR with the fix, otherwise I am going to fix it later today.

astrikos commented 6 years ago

Addressed in https://github.com/astrikos/wetransfer-python-sdk/commit/5b961e48d175c66be040eb1f8e4bf1d1e80c6f11. Closing now thanks for the report!