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

[Feature] Option to store logfile at application directory instead into AppData #371

Closed
teamayu opened this issue Apr 1, 2022 · 5 comments
Assignees

Comments

@teamayu
Copy link

teamayu commented Apr 1, 2022

Currently running Windows 11. Really nice software, but why does it write the logs and config file to:
AppData/Roaming/SystemTrayMenu/*

?

@Hofknecht
Copy link
Owner

@teamayu
thanks for the question

might be related to
#120

maybe also this link
https://stackoverflow.com/questions/982354/where-are-the-properties-settings-default-stored#:~:text=If%20you%20store%20settings%20for,public%5Cappdata%5Clocal%20

i think it was when we put it in the app store there was the issue that logfile could not be written at same location as the exe

config file can be written into assembly location at portable version to import and export config
2022-04-01 21_02_06-Window

At which place would you have expected the log and the config?

@Hofknecht
Copy link
Owner

You can also open the log here, right click on the system tray icon

2022-04-01 21_05_30-

@teamayu
Copy link
Author

teamayu commented Apr 2, 2022

Thanks for the info. Yes, it would be ideal if the data could be written to the same folder that the software is run from. Not a deal breaker though.

@Hofknecht
Copy link
Owner

i think we will implement to store logfile at assembly location and we will try to set this and store config at assembly location as default in the portable version

@Hofknecht Hofknecht self-assigned this Apr 15, 2022
@Hofknecht Hofknecht changed the title Writes to AppData Folder [Feature] Writes to AppData Folder May 2, 2022
@Hofknecht Hofknecht changed the title [Feature] Writes to AppData Folder [Feature] Option to store logfile at application directory instead into AppData Jun 5, 2022
Hofknecht added a commit that referenced this issue Jun 5, 2022
@Hofknecht
Copy link
Owner

Implemented in version 1.2.9.11

Some notes:
is also default false for portable version because it is possible that you only have read rights also in the portable app.

If you switch the setting on and have no rights when applying the setting, exiting the app or when starting the app, it is automatically switched off again and logged into logfile that there are no rights

also fixed that current logfile is shown on top if there are 2 logfiles

also fixed if the log file gets too big, a backup is made and a new log file is started, if that goes wrong, it was not logged

moved a bit the layout e.g. 'Use icon from directory' moved from General to Customize tab

Hofknecht added a commit that referenced this issue Jun 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants