Skip to content

Transmit React client.js warnings/errors to Python logger #816

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

Closed
Archmonger opened this issue Oct 5, 2022 · 2 comments
Closed

Transmit React client.js warnings/errors to Python logger #816

Archmonger opened this issue Oct 5, 2022 · 2 comments
Labels
priority-3-low May be resolved one any timeline.

Comments

@Archmonger
Copy link
Contributor

Current Situation

Currently, all warnings or exceptions generated within the react client are only displayed on the front-end. There is a reasonable expectation that most users aren't always going to be looking at inspect tools. Thus, when developing problematic IDOM code there is often no visual indication of what issues arise, leading to a confusing developer experience.

For example, warnings like client.js:3636 Warning: Invalid DOM property `class`. Did you mean `className`? should be transmitted to Python.

The fact that these issues aren't displayed in Python feels a bit disjointed for a Python web development framework.

Proposed Actions

Catch all warnings/exceptions from client.js and transmit them to a Python logger via websockets.

@Archmonger Archmonger added type: feature priority-3-low May be resolved one any timeline. labels Oct 5, 2022
@Archmonger Archmonger added this to the 2.0 milestone Oct 5, 2022
@rmorshea
Copy link
Collaborator

rmorshea commented Oct 6, 2022

Related to #346

@rmorshea rmorshea removed this from the Luxury milestone Feb 21, 2023
@Archmonger
Copy link
Contributor Author

Closing in favor of #890

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority-3-low May be resolved one any timeline.
Projects
None yet
Development

No branches or pull requests

2 participants