Added handling and counters for filename, http, and time errors

KGBicheno commented 3 years ago

I've added extra error-handling for:

Situations where a block's parent returns NoneType. If so, it now checks for a valid child element to replace it and fails that block if none are found. The program will, however, continue past that block.
Some blocks were returning a HTTPError in the Notion API through client.py at the point where enhancer.py tried to load the block using NotionID. An error at this point will now fail that block but allow for the program to continue.
The creation of the Zip file was failing in situations where the lastEditedTime tuple returned a NoneType error. I attempted to have the program replace missing create times with edited times and vice versa, but the issue was with the tuple, and I couldn't see where the error was being introduced. The block will still load, but with the default (1980, 1, 1, 0, 0, 0) timestamp attached.

The final output will now include the number of times any of these errors occurred.

KGBicheno commented 3 years ago

There's a bug where hitting enter without an input will still loop the question, but it will crash no matter what you enter next. Otherwise it's fine. Run through 3.6.8 as per instructions.

Pushed mainly to show you where I got up to. I'm going to redo the whole thing in a day or two. It would probably be easier to add a flag for setting the options at the start - I had hundreds of inputs from my Notion pages. :D Thanks for letting me have a go! :dancers:

Cobertos commented 3 years ago

Going to close this one, I merged a couple things from here into https://github.com/Cobertos/notion_export_enhancer/commit/8c72109d44e356287c680b76be2e96a5bcc40dfc

Thanks again for the work and sorry I haven't been more communicative! This just hasn't been a priority for me

Cobertos / notion_export_enhancer

Added handling and counters for filename, http, and time errors #2