Closed aidaner closed 2 years ago
This pull request introduces 1 alert when merging 9d230be951a826c32cf3c6b70516e91598d31b55 into 3ef863d1495da0ec4d598ed1054474298e4c367f - view on LGTM.com
new alerts:
Cool! I will merge this with next version but before that, I need to test this...
ok
ok. i will change it back
Thank you so much! I will test this and merge it. BTW What's your Scratch Username? I want that because I want to thank you in credits section of Readme...
My scratch username is AidanER1
My scratch username is AidanER1
Thank you! I will add that username in credits...
i changed the events to get updates from the websocket like the browsers do instead of sending a request to the api every time. but the websocket does not send the username or timestamp when a new change occurs so i replaced it with a None. i can revert this if you want. and i fixed the broken pipe error by making a new thread that pings the websocket so the server does not close the connection and the thread auto closes when the main script is done