kingy444
kingy444
the way i read the config screp, if i removed the lines mentioned i dont think it will actually even touch your current config, does this sound right to you?
I have the opposite actually - I have the Siren and Spotlight flags in my 822a and the camera doesn’t support those
My bad misread that - not sure why the 822 is reporting capability it doesn’t have
I have reviewed all codacy bot detections and corrected where possible. For info. Most of the detections are related to the fact that the Control.FullScreen.js file is a plugin, and...
Will work through it and update this once done. Looks like the Spidery is not on cdnjs atm so ill see if we can get it up there
Thanks for the tip - added **# noqa: E501** to the cdnjs affected lines - tests now passing
Unfortunately new to this github thing - might need a bit more to go on than that. How can You submit smaller pull requests when all changes are in the...
I think i did that right - there are definitely a lot less entries in the PR. Let me know if i need to do anything else.
unfortunately most of the fixes are part of the openvpn-monitor.py file so breaking them out is quite difficult. i did most of the changes on my local before forking. what...
Evening Mate - I have obviously struggled to get this broken into smaller commits, mainly due to all the changes being in the one file. If need be I will...