Skip to content

Conversation

@TheBakedPotato
Copy link
Collaborator

The i command for the GPAD API now returns the version of the GPAD API itself.

Instead of using a preprocessor directive for the version value, it is now an actual member of the new GPAD_API class. This class, while very bare at the moment, can also be used to better contain the the methods/functionality inside the gpad_hal.h/c source files.

@TheBakedPotato TheBakedPotato force-pushed the 206-i-command-return-protocol-version branch from c79c4bc to 7084f89 Compare December 8, 2025 01:05
@TheBakedPotato TheBakedPotato force-pushed the 273-move-to-platformio branch 2 times, most recently from b74c930 to cd2497a Compare December 11, 2025 23:35
@TheBakedPotato TheBakedPotato force-pushed the 206-i-command-return-protocol-version branch from 7084f89 to d33c773 Compare December 11, 2025 23:37
Base automatically changed from 273-move-to-platformio to main December 15, 2025 19:08
- Added a char enum which maps the chars used to indicate the
  commands to actual enum variants
- The main switch statement in GPAD_HAL.cpp now switches over the
  Command enum instead of the hardcoded chars
- GPAD_API_VERSION is now defined as a preprocessor directive, like
  FIRMWARE_VERSION
- The value is published via MQTT just like the FIRMWARE_VERSION
…ment

- The GPAD API version is now hardcoded as an actual value
- It is not something that should be able to be changed by a compilation flag
@TheBakedPotato TheBakedPotato force-pushed the 206-i-command-return-protocol-version branch from acc73aa to ae6b969 Compare December 17, 2025 18:16
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