Skip to content
This repository was archived by the owner on Feb 9, 2026. It is now read-only.

Equipment Support - #5

Closed
jeremy-green wants to merge 6 commits into
iamkarlson:mainfrom
jeremy-green:feature/equipment-support
Closed

jeremy-green wants to merge 6 commits into
iamkarlson:mainfrom
jeremy-green:feature/equipment-support

Conversation

@jeremy-green

@jeremy-green jeremy-green commented Mar 23, 2025 •

Copy link
Copy Markdown

Description

This commit adds support for Grocy's Equipment feature to the PyGrocy2 library:

  • Add Equipment data model (data_models/equipment.py)
  • Add Equipment API client methods in grocy_api_client.py
  • Add Equipment methods to Grocy class
  • Add unit tests for Equipment model
  • Update imports in test files

Checklist

  • The code change is tested and works locally.
  • tests pass. Your PR won't be merged unless tests pass
  • There is no commented out code in this PR

This commit adds support for Grocy's Equipment feature to the PyGrocy2 library:

- Add Equipment data model (data_models/equipment.py)
- Add Equipment API client methods in grocy_api_client.py
- Add Equipment methods to Grocy class
- Add unit tests for Equipment model
- Update imports in test files
This commit removes the code and properties related to equipment maintenance since
this functionality is not currently implemented in Grocy. The implementation now
focuses solely on the equipment properties currently available in the Grocy API.

Changes:
- Remove maintenance fields from EquipmentDetailsResponse
- Remove maintenance properties from Equipment class
- Update tests to reflect the simplified implementation
- Clean up code comments that referenced future functionality
@iamkarlson
iamkarlson changed the base branch from develop to main January 8, 2026 10:02
@iamkarlson

Copy link
Copy Markdown
Owner

Hi @jeremy-green I had to change base branch for your MR because I changed default branch and a few other dev stuff in this repo. I will review your PR now! :D

@iamkarlson

Copy link
Copy Markdown
Owner

Merged as part of #12

@iamkarlson iamkarlson closed this Feb 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants