search

fgebhart / zeit-on-tolino

Service to auto-upload the ZEIT 🗞 e-paper to your tolino cloud library 📚
MIT License
14 stars 23 forks source link

do not treat "file already exists" as an error to prevent error notifications #25

Closed superhenne closed 1 year ago

superhenne commented 2 years ago

When the .epub file that the script is trying to upload to webreader.mytolino.com already exists, an error is generated (FileExistsError). This leads to error notifications sent out by github. IMO this is not a real error, and should rather be ignored. At least no notification should be generated. Alternatively, there is a way to prevent further executions of the workflow once the upload has been successfully run.

fgebhart commented 1 year ago

Thanks for filing this issue. Sounds like an valuable addition to me. I will try to see what it would take to implement this.

fgebhart commented 1 year ago

The linked PR implements the requested enhancement. We should be able to verify the impact on Wednesday.