Skip to content

feat(backend): add logger destination on configuration#393

Closed
JavierRibaldelRio wants to merge 3 commits intocontrol-station/electronfrom
control-station/electron-configs
Closed

feat(backend): add logger destination on configuration#393
JavierRibaldelRio wants to merge 3 commits intocontrol-station/electronfrom
control-station/electron-configs

Conversation

@JavierRibaldelRio
Copy link
Contributor

@JavierRibaldelRio JavierRibaldelRio commented Nov 26, 2025

Modify pkg/logger and configuration to allow users to specify a path to store the logs.

Required to store the logs using electron.

Partially addresses #225


I also found out that due to the update of theadj (commit commit ab3a836 [approved on #388]) packet sender has stop working because excepts to find adj at its previous location software/backend/cmd/adj but is no longer there.

@JavierRibaldelRio JavierRibaldelRio added Backend go Pull requests that update go code labels Nov 26, 2025
@maximka76667
Copy link
Contributor

maximka76667 commented Nov 27, 2025

Could you change both config.toml and dev-config.toml to have the same structure (with same [table-names] and same settings, but with corresponding values for each of them)?
Also I think it would be better to change [logging] TOML table to [logger].

@JavierRibaldelRio
Copy link
Contributor Author

Closed to do merge on main instead of control-station/electron. See #394.

@JavierRibaldelRio JavierRibaldelRio changed the title feat(backend): add logger destination on configurarion feat(backend): add logger destination on configuration Nov 27, 2025
@Humanoidear Humanoidear deleted the control-station/electron-configs branch November 27, 2025 09:35
@JavierRibaldelRio
Copy link
Contributor Author

Could you change both config.toml and dev-config.toml to have the same structure (with same [table-names] and same settings, but with corresponding values for each of them)?

@maximka76667, No on dev-config.toml there are specific options of that aren't used on config.toml

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backend go Pull requests that update go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants