Move background action console.log inside platforms check#9891
Move background action console.log inside platforms check#9891linonetwo wants to merge 5 commits into
Conversation
✅ Deploy Preview for tiddlywiki-previews ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
📊 Build Size Comparison:
|
| Branch | Size |
|---|---|
| Base (master) | 2491.8 KB |
| PR | 2491.8 KB |
Diff: ⬆️ Increase: +0.0 KB
✅ Change Note Status
All change notes are properly formatted and validated!
📝 $:/changenotes/5.5.0/#9891
Type: bugfix | Category: internal
Release: 5.5.0
Move background action log inside platforms check to avoid spurious server-side logs
🔗 #9891
👥 Contributors: linonetwo
📖 Change Note Guidelines
Change notes help track and communicate changes effectively. See the full documentation for details.
|
Confirmed: linonetwo has already signed the Contributor License Agreement (see contributing.md) |
There was a problem hiding this comment.
Pull request overview
This PR adjusts background-action processing logging so it only occurs when actions are actually executed (i.e., after the platform gating logic decides doActions is true). This reduces log noise when background actions are suppressed due to a platforms field mismatch.
Changes:
- Move the
"Processing background action"console.logto run only inside thedoActionsbranch.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@linonetwo could you please update the release note to target v5.5.0? Thank you. |
PR TiddlyWiki#9891 is still open, so the background-action log change is planned for 5.5.0 instead of 5.4.1.
is not a valid category; use instead.
I found this cosole log very annoying, because I already set it to only run on the browser, but it still create a lot of log in the server side.
Now, I make it only log when it's going to really execute it.