Just created a PR with two main improvements to the code:
Removed the close_browser method and created the more efficient close_when_message_successfully_sent method, making it useful to close the browser when the message is successfully sent.
Added CLI logging capability, implemented as a replacement to print errors. Professional approach, expanding its possibilities to further handlers if needed (for example, logging not only into the stream but into a file, through a TCP socket, or even meshed with SysLog if necessary - please, refer to the logging package docs for more info, it's REALLY useful).
Added multiline support, as per issue #42.
Over my free time in the following days, I'll be working on the identification of opened browser instances and their usage instead of a mandatory "no other opened browser" requirement for a nice and smooth execution of the code.
PS: @Kalebu, please check your Twitter inbox, sent you some info that may be of your interest.
Hiya, folks.
Just created a PR with two main improvements to the code:
close_browser
method and created the more efficientclose_when_message_successfully_sent
method, making it useful to close the browser when the message is successfully sent.print
errors. Professional approach, expanding its possibilities to further handlers if needed (for example, logging not only into the stream but into a file, through a TCP socket, or even meshed with SysLog if necessary - please, refer to the logging package docs for more info, it's REALLY useful).Over my free time in the following days, I'll be working on the identification of opened browser instances and their usage instead of a mandatory "no other opened browser" requirement for a nice and smooth execution of the code.
PS: @Kalebu, please check your Twitter inbox, sent you some info that may be of your interest.
nCKbr.