Skip to content

feature: add a flag for recording extra activity logs.#9

Merged
chrisdoehring merged 1 commit intomainfrom
release-20260309-hotfix
Mar 11, 2026
Merged

feature: add a flag for recording extra activity logs.#9
chrisdoehring merged 1 commit intomainfrom
release-20260309-hotfix

Conversation

@chrisdoehring
Copy link
Copy Markdown

This pull request introduces improvements to documentation, configuration, and logging behavior for the Ctrack Crystal integration. The main changes include adding a comprehensive guide for developers, enhancing configuration options for the pull observations action, and refining logging logic to provide more control over activity records.

Documentation and Developer Guidance:

  • Added a detailed CLAUDE.md file that explains project architecture, commands for development and testing, and conventions for working with the codebase. This guide is intended to help developers (including AI assistants) understand and contribute to the repository more effectively.

Configuration Enhancements:

  • Added docstrings to the AuthenticateConfig and PullObservationsConfig classes in app/actions/configurations.py to clarify their purpose and usage. [1] [2]
  • Introduced a new configuration option, log_all_vehicle_activity, to PullObservationsConfig. This option allows users to control whether activity records are logged for every vehicle, even when no new observations are found.

Logging Behavior Improvements:

  • Updated the per-vehicle logging logic in app/actions/handlers.py to respect the new log_all_vehicle_activity configuration. Now, activity is logged for all vehicles only if this option is enabled, otherwise only when new observations are present.

@chrisdoehring chrisdoehring merged commit f0cd9a4 into main Mar 11, 2026
7 checks passed
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.

1 participant