trent_larson/crowd-funder-for-time-pwa
4
1
Fork 2
Code Issues Pull Requests 22 Releases 5 Wiki Activity

feat: centralize debug logging with environment control #151

Closed
anomalist wants to merge 1 commits from logging-upgrade into build-improvement
pull from: logging-upgrade
merge into: trent_larson:build-improvement
Conversation 0 Commits 1 Files Changed 15 +223 -47
anomalist commented 2025-07-31 03:34:40 +00:00
Owner
Copy Link
  • Replace stray console.log statements with logger.debug() for controlled output
  • Add VITE_DEBUG_LOGGING environment variable control for all debug messages
  • Convert databaseUtil, PlatformServiceFactory, and WebPlatformService to use centralized logger
  • Wrap Electron console.log statements with environment variable checks
  • Update SQLWorker to use controlled debug messaging instead of direct console.log
  • Ensure debug logging is disabled by default for production safety

All debug logging now controlled by VITE_DEBUG_LOGGING=true environment variable.

- Replace stray console.log statements with logger.debug() for controlled output - Add VITE_DEBUG_LOGGING environment variable control for all debug messages - Convert databaseUtil, PlatformServiceFactory, and WebPlatformService to use centralized logger - Wrap Electron console.log statements with environment variable checks - Update SQLWorker to use controlled debug messaging instead of direct console.log - Ensure debug logging is disabled by default for production safety All debug logging now controlled by VITE_DEBUG_LOGGING=true environment variable.
👍 1
anomalist added 1 commit 2025-07-31 03:34:41 +00:00
feat: centralize debug logging with environment control 6a622d20b8 
- Replace stray console.log statements with logger.debug() for controlled output
- Add VITE_DEBUG_LOGGING environment variable control for all debug messages
- Convert databaseUtil, PlatformServiceFactory, and WebPlatformService to use centralized logger
- Wrap Electron console.log statements with environment variable checks
- Update SQLWorker to use controlled debug messaging instead of direct console.log
- Ensure debug logging is disabled by default for production safety

All debug logging now controlled by VITE_DEBUG_LOGGING=true environment variable.
anomalist closed this pull request 2025-08-14 02:04:20 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
anomalist jose jsnbuchanan kentbull trentlarson
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: trent_larson/crowd-funder-for-time-pwa#151
Delete Branch "logging-upgrade"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?