Skip to content

fix: make logs easier to read#1065

Merged
sidvishnoi merged 1 commit into
mainfrom
logging-improvements
May 29, 2025
Merged

fix: make logs easier to read#1065
sidvishnoi merged 1 commit into
mainfrom
logging-improvements

Conversation

@sidvishnoi
Copy link
Copy Markdown
Member

Context

Extracted from #1064

Changes proposed in this pull request

  • If loggerName includes a /, show the content after / differently so it doesn't take too much attention (or look noisy)
  • In background's message listener log, log message.action followed by message.payload, instead of logging message as an object which needs clicking to expand its contents.
  • Avoid logging when switching windows when there are no tabIds to pause monetization for.
Image

@sidvishnoi sidvishnoi requested a review from DarianM May 29, 2025 14:22
@github-actions github-actions Bot added area: background Improvements or additions to extension background script area: shared Changes to shared libraries and utilities labels May 29, 2025
@github-actions
Copy link
Copy Markdown
Contributor

Extension builds preview

Name Link
Latest commit 9cccc4c
Latest job logs Run #15326059650
Chrome (990.5KB)Download
Firefox (990.53KB)Download

@sidvishnoi sidvishnoi merged commit 772e5ef into main May 29, 2025
9 checks passed
@sidvishnoi sidvishnoi deleted the logging-improvements branch May 29, 2025 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: background Improvements or additions to extension background script area: shared Changes to shared libraries and utilities

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants