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

feat(logging): implement configurable log levels via VITE_LOG_LEVEL #161

Open
anomalist wants to merge 0 commits from logger-level into master
pull from: logger-level
merge into: trent_larson:master
Conversation 1 Commits - Files Changed -
anomalist commented 2025-08-12 06:09:04 +00:00
Owner
Copy Link

Add comprehensive logging configuration system with environment variable support.
Environment files now include appropriate log levels per build mode:

  • Development: debug (maximum visibility)
  • Production: warn (minimal noise)
  • Testing: info (balanced output)

Includes smart default behavior based on platform and environment,
enhanced logger methods for level checking, and comprehensive documentation.
All existing logging calls remain backward compatible.

Closes logging configuration request

Add comprehensive logging configuration system with environment variable support. Environment files now include appropriate log levels per build mode: - Development: debug (maximum visibility) - Production: warn (minimal noise) - Testing: info (balanced output) Includes smart default behavior based on platform and environment, enhanced logger methods for level checking, and comprehensive documentation. All existing logging calls remain backward compatible. Closes logging configuration request
anomalist added 1 commit 2025-08-12 06:09:05 +00:00
feat(logging): implement configurable log levels via VITE_LOG_LEVEL d30597a921 
Add comprehensive logging configuration system with environment variable support.
Environment files now include appropriate log levels per build mode:
- Development: debug (maximum visibility)
- Production: warn (minimal noise)
- Testing: info (balanced output)

Includes smart default behavior based on platform and environment,
enhanced logger methods for level checking, and comprehensive documentation.
All existing logging calls remain backward compatible.

Closes logging configuration request
anomalist added 1 commit 2025-08-12 06:26:08 +00:00
fix(build): resolve shell script export error in .env.development 7df52312ba 
Fixed malformed comment in .env.development that was causing
"export: production).=: not a valid identifier" error in build scripts.
The comment was split across lines, causing the shell to interpret
" production)." as a variable assignment.

- Removed malformed comment line that was breaking build:web script
- Build script now successfully validates environment and starts server
- Resolves issue preventing Playwright tests from running properly
trentlarson commented 2025-08-14 02:06:23 +00:00
Owner
Copy Link

👍

👍
anomalist added 10 commits 2025-08-14 05:39:26 +00:00
chore: lint-fix 2afe61d752
refactor(ui): remove unused image cache system and fix TypeScript compilation bb357f294a 
- Remove imageCache Map that only stored null values
- Remove selectedImageData Blob property (never used)
- Remove cacheImageData method and related function props
- Remove handleImageLoad method from ActivityListItem
- Clean up ImageViewer component props
- Fix TypeScript compilation by replacing legacy logConsoleAndDb calls
- Replace logConsoleAndDb with logger.error in deepLinks service
- Images continue to work via direct URL references

The image cache system was non-functional and only stored null values.
ImageViewer component ignored blob data and only used URLs.
Fixed production build failure caused by undefined logConsoleAndDb function.
Removing dead code improves maintainability without affecting functionality.
fix(typescript): resolve production build errors and add ESLint ignore comments 8c0b547855 
- Fix TypeScript compilation errors in deepLinks service by replacing logConsoleAndDb with logger.error
- Add ESLint disable comments for necessary 'any' type usage in worker polyfills and Vue mixins
- Add ESLint disable comments for console statements in test files and debugging code
- Production build now succeeds with npm run build:web:prod
- TypeScript compilation passes with npm run type-check

The deepLinks service was using undefined logConsoleAndDb function causing build failures.
Worker context polyfills and Vue mixin complexity require 'any' type usage in specific cases.
Console statements in test files and debugging code are intentionally used for development.
fix(lint): resolve low and medium impact warnings adfaef7947 
- Remove console statements and replace with proper logging
- Fix line length violations in comments
- Maintain functionality while improving code quality

Reduces lint warnings from 30 to 25 by addressing:
- 3 console statement violations
- 2 line length violations
doc: Add helpful setup instructions for quick start for new devs. 81377d1fb6
chore: Bump version and add "-beta". 24a2dce43e
feat: Add easier way for test users to register themselves. a221a5c5ed
doc: Add instructions to become test user, and other README refactors. 81096a8bee
Merge branch 'master' into remove-image-cache a3b69fc0d4
Merge branch 'master' into logger-level c91ac135b3
This branch is already included in the target branch. There is nothing to merge.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin logger-level:logger-level
git checkout logger-level
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
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: trent_larson/crowd-funder-for-time-pwa#161
Delete Branch "logger-level"

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?