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

[Issue]: Improper error handling #134

Open
AlvinAtThales opened this issue Jul 21, 2023 · 1 comment
Open

[Issue]: Improper error handling #134

AlvinAtThales opened this issue Jul 21, 2023 · 1 comment
Assignees
Labels
invalid This doesn't seem right medium Medium priority

Comments

@AlvinAtThales
Copy link
Contributor

Issue Description

Errors caught in the main process (backend) are not sent to the renderer process (frontend) which cause frontend code to continue executing despite the error. This issue has been present since version 1.0

Duplicate Steps

No response

Expected Results

Errors caught in main process should be processed and sent to the renderer process for it to be caught and display the error message

Version Detected

1.0.1

Attachments

No response

@AlvinAtThales AlvinAtThales added invalid This doesn't seem right medium Medium priority labels Jul 21, 2023
@AlvinAtThales AlvinAtThales self-assigned this Jul 21, 2023
@CarlatThales
Copy link
Contributor

This includes a study on the logs:

  • Understand functionality of logging available in Electron apps without dependencies, does it support different logging levels
  • When there is an error ideally the logs should be printed with error level and this should be on by default.
    ** By default info, when parsing or calculating the equations should be at INFO level

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right medium Medium priority
Projects
None yet
Development

No branches or pull requests

3 participants