Lyndon1994 / EfficientOfficeTools

Wox Plugins, Alfred Workflows, Chrome Extensions
MIT License
15 stars 1 forks source link

MyWebHistory | browserhistory.py and main.py #2

Open codylellis opened 5 months ago

codylellis commented 5 months ago

browserhistory.py

Line 148, creating a backup of history file I would recommend removing the if statement. I didn't completely debug it, but I can never get new history out of the plugin unless this line is removed.

main.py

Line 35: if qs[0] == 'vsc':

I would remove this because it generates a error in the error.log for index out of range, and don't get exact line error from logging module. would add Log.error(traceback.format_exc()) under line 69

say something like the below to evaluate first

if 'vsc' in qs:

Lyndon1994 commented 5 months ago

Thanks for your using and response, please feel free to commit a PR