Skip to content

fix: Add missing log_system_event convenience function#115

Merged
alireza787b merged 1 commit intomain-candidatefrom
claude/backend-review-polish-01NQ3abACnaVRDY1WgD7bKen
Nov 22, 2025
Merged

fix: Add missing log_system_event convenience function#115
alireza787b merged 1 commit intomain-candidatefrom
claude/backend-review-polish-01NQ3abACnaVRDY1WgD7bKen

Conversation

@alireza787b
Copy link
Owner

Add log_system_event to module-level convenience functions in logging_config.py to match log_system_error and log_system_warning pattern.

Resolves: ImportError: cannot import name 'log_system_event' from 'logging_config'

Pull Request

Description

Type of Change

  • Bug fix (non-breaking change fixing an issue)
  • New feature (non-breaking change adding functionality)
  • Breaking change (fix or feature causing existing functionality to not work as expected)
  • Documentation update
  • Performance improvement
  • Code refactoring
  • Configuration change

Related Issues

Fixes #
Related to #

Changes Made

Testing

SITL Testing

  • Tested in SITL environment
  • Number of drones tested:
  • Test scenarios:

Code Quality

  • Python syntax validated
  • Frontend builds without errors
  • No new ESLint warnings (or documented)

Documentation

  • Documentation updated (if needed)
  • CHANGELOG.md updated (if applicable)

Screenshots/Videos

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have tested my changes in SITL environment
  • Any dependent changes have been merged and published

Additional Notes


Note for Maintainers:

  • Target branch should be main-candidate for normal PRs
  • Only merge to main for release preparation

Add log_system_event to module-level convenience functions in logging_config.py
to match log_system_error and log_system_warning pattern.

Resolves: ImportError: cannot import name 'log_system_event' from 'logging_config'
@alireza787b alireza787b merged commit 1259816 into main-candidate Nov 22, 2025
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants