civisanalytics / civis-python

Civis API Python Client
BSD 3-Clause "New" or "Revised" License
34 stars 26 forks source link

ENH more informative error for expired files #337

Closed elsander closed 4 years ago

elsander commented 4 years ago

Raise a more informative exception when calling civis.io.file_to_dataframe on a file with no url (which likely means the file is expired). I went with a ValueError here but could change it if a different exception would be easier to understand. This error check is pre-empting a ValueError, since the error comes from reading a csv from None.

Closes #336.