Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Logging improvements for the Python code #196

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

md8n
Copy link
Contributor

@md8n md8n commented Mar 3, 2023

Description

Literally just logging improvements for the Python/Flask code to make the mesages clearly show where they are coming from.

Plus, a pre-emptive addition to .gitignore for node and a better initialization of UIProcessor.py

... and that's all ...

@md8n md8n requested a review from Orob-Maslow March 3, 2023 05:43
@md8n md8n self-assigned this Mar 3, 2023
@Orob-Maslow
Copy link
Contributor

Logging in the browser or in the log.txt file? I don't see anything other than the error on the console saying socketio is a problem:
The client is using an unsupported version of the Socket.IO or Engine.IO protocols (further occurrences of this error will be logged with level INFO)

and it matches the web site error

Screenshot from 2023-03-03 18-03-57

but the menus don't work, so not sure if this is what you wanted to pull in as a PR, but I hesitate because it does not function. Using chrome on 20.04 ubuntu.

@md8n
Copy link
Contributor Author

md8n commented Mar 4, 2023

Logging in the browser or in the log.txt file? I don't see anything other than the error on the console saying socketio is a problem: The client is using an unsupported version of the Socket.IO or Engine.IO protocols (further occurrences of this error will be logged with level INFO)

and it matches the web site error

but the menus don't work, so not sure if this is what you wanted to pull in as a PR, but I hesitate because it does not function. Using chrome on 20.04 ubuntu.

Correct it doesn't work - but unfortunately neither does the master branch at this moment, because my earlier :pr: that bumped all of the python dependencies got merged in before I realised that I hadn't updated the matching javascript dependencies

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants