Skip to content

Conversation

@peppescg
Copy link
Collaborator

@peppescg peppescg commented Jul 4, 2025

Used electron-log replacing console with log fn.

The log will be saved into the filesystem:

  • on Linux: ~/.config/{app name}/logs/main.log
  • on macOS: ~/Library/Logs/{app name}/main.log
  • on Windows: %USERPROFILE%\AppData\Roaming{app name}\logs\main.log

nit: we can improve or replace later, but we got some issues using directly the dmg download from github repo, that we cannot reproduce locally, so it would helpful to have some additional info here.

Logs with info|warn|error

Screenshot 2025-07-04 at 15 09 55

Logs in both thv serve and logs file

Screenshot 2025-07-04 at 15 07 05

@peppescg peppescg self-assigned this Jul 4, 2025
@peppescg peppescg changed the title Issues/418 chore(log): add electron-log in order to troubleshooting possible errors Jul 4, 2025
@peppescg peppescg changed the title chore(log): add electron-log in order to troubleshooting possible errors chore(log): add electron-log in order to troubleshooting issues Jul 4, 2025
@peppescg peppescg linked an issue Jul 4, 2025 that may be closed by this pull request
Copy link
Collaborator

@samuv samuv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice! just a comment on the bump

kantord
kantord previously approved these changes Jul 4, 2025
samuv
samuv previously approved these changes Jul 4, 2025
Copy link
Collaborator

@samuv samuv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💪🏼 thank you, it should be much easier to debug now

@peppescg peppescg dismissed stale reviews from samuv and kantord via 283159c July 4, 2025 13:55
@peppescg peppescg requested review from kantord and samuv July 4, 2025 14:00
@peppescg peppescg merged commit 52acab8 into main Jul 4, 2025
11 checks passed
@peppescg peppescg deleted the issues/418 branch July 4, 2025 14:03
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.

[Task] Add logs in node context

4 participants