Conversation
- refactored runners into __main__ - consistent implementation for cli and mcp - TODO! should move the tui to do the same
post this installed everything using uv sync --all-extras
- introduces cc/APIClient - moves discovery to be sync (used to be async) - makes everything in APIEndpoint instance level - introduces a config class based on pydantic-settings - introduces a header mixin (see comments for details) - many static methods are now instance methods refs #96
- get_config is now an instance method - get_config is called when the individual clients are initalised - discover is no longer referenced on each method call
- use this as the reference implementation - see changes to load_cert_chain method - see changes to temp_file_private_key.flush()
- tests are still failing don't deploy yet
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Adds complete SDK, DTO, test, and CLI coverage for all remaining discovered-but-unimplemented Gallagher API features: alarm_zones, competencies, elevators, fence_zones, inputs, interlock_groups, macros, and outputs. Each endpoint follows the established pattern of Ref/Summary/Detail/Response DTOs, an APIEndpoint subclass wired into APIClient, a pytest fixture + list/detail test file, and list/get CLI subcommands. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- don't want this visible in pypi - fixes issue with running tests on @github actions
- i have no idea how i missed this - writes about pdf and personal_data_fields in card detail
see documentation as to why this should not be done Each response will also contain a next link that will take you to the non-blocking version of the call at /api/events. However if you are using this call you should not also be using the next link: applications typically use one or the other, not both.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.