Skip to content

Hot Fixes + New Settings + Code Refactoring#77

Merged
ToxicStoxm merged 15 commits intomainfrom
dev
Aug 4, 2024
Merged

Hot Fixes + New Settings + Code Refactoring#77
ToxicStoxm merged 15 commits intomainfrom
dev

Conversation

@ToxicStoxm
Copy link
Owner

@ToxicStoxm ToxicStoxm commented Aug 4, 2024

Bug Fixes include:

  • Temporarily fixed some CLI arguments producing unwanted side effects by simply removing them for now.
  • Fixed Network manager not reconnecting or getting stuck in various states with additional checks and minimal changes to the socket reboot and reconnection logic.
  • Fixed Logger displaying incorrect metadata by adjusting the trace depth dynamically.

New Settings:

  • Log Level Colors are now settable.
  • Stack trace depth is now settable.
  • Option to disable logger color coding entirely.

Code Refactoring:

  • Logger cleanup.
  • Log file processing is now split up into multiple functions.

Other:

  • Minimal README and Flathub config changes by @hannescam.

ToxicStoxm and others added 15 commits July 29, 2024 19:42
@ToxicStoxm
Merge pull request #73 from ToxicStoxm/main
da9688c 
Catch up to main
@ToxicStoxm
- documentation (15%)
66ed125 
- code cleanup (15%)
@ToxicStoxm
- documentation (15%)
0f9ac30 
- code cleanup (15%)
@ToxicStoxm
- documentation (30%)
d9df222 
- code cleanup (20%)
@hannescam
Merge pull request #75 from ToxicStoxm/hannescam-patch-2
6f24aa3 
Add new backend link
@ToxicStoxm
Fixed bug(s) which caused the application to crash when changing host…
9bc0ea8 
… or just blocking the UI. Added toast to inform user about reconnection to previous host after new connection failed
@ToxicStoxm
Merge remote-tracking branch 'origin/dev' into dev
73eec61
@ToxicStoxm
Bug fixes
a01fea8 
- Fixed application not reconnecting to server automatically
- Fixed application stuck in disconnected state
- Fixed application not correctly reconnecting after connection to a new server failed
- Fixed logger displaying the wrong metadata (Dynamic stack trace depth)
@ToxicStoxm
Cleanup
7bf3c8d
@ToxicStoxm
Merge pull request #76 from ToxicStoxm/main
bd57b65 
Catch up dev to main
Added developer ID and made the description a bit more comprehensive
28bde1d
@ToxicStoxm
Bug fixes and new settings
6e64a42 
Implemented a few new settings including log color customization, stack-trace-depth.
Disables some CLI arguments since they introduced bugs.
Added test for in18
Added new log level STACKTRACE
Fixed application spamming console when disconnected from the server
Refactored some parts of the Logger
@ToxicStoxm
Documentation
f27c9fb 
Added comments for clarification in config.yaml for the newly added settings, from the last commit.
@ToxicStoxm
Refactoring, Cleanup, Comments
3b26694 
- Added some comments for clarification
- Moved the log file processing out of the logic_init function to make the code easier to read
- Minimal Code Cleanup
@ToxicStoxm
Merge remote-tracking branch 'origin/dev' into dev
b0d29f4
@ToxicStoxm ToxicStoxm merged commit a00173a into main Aug 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ToxicStoxm @hannescam