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

feat: Enhance log directory name #8376

Merged
merged 1 commit into from
Nov 26, 2024
Merged

feat: Enhance log directory name #8376

merged 1 commit into from
Nov 26, 2024

Conversation

thisisamir98
Copy link
Contributor

@thisisamir98 thisisamir98 commented Nov 26, 2024

Description:

This change improves the naming convention for log directories by including a human-readable date and time in the folder name.

Why it's useful:

Previously, the directories were named using non-descriptive identifiers like webViewId, which made it difficult to quickly identify when logs were created. This becomes especially problematic when users download debug logs and send a zip file containing numerous directories. The lack of context in the folder names forces extra effort to match logs to their respective events or timelines.

By including a clear timestamp, each folder now provides immediate insight into when it was created, making it easier to navigate, analyze, and debug issues effectively.

Before:
image

After (date & time added):
image

Copy link

sonarcloud bot commented Nov 26, 2024

@thisisamir98 thisisamir98 merged commit bcf26a4 into dev Nov 26, 2024
7 checks passed
@thisisamir98 thisisamir98 deleted the better-log-dir-name branch November 26, 2024 11:27
@paulwire paulwire added the echoes: technical-roadmap/technical-debt More specific category, to highlight Technical Debt being tackled. label Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
echoes: technical-roadmap/technical-debt More specific category, to highlight Technical Debt being tackled. type: feature / request ✨
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants