Skip to content
This repository has been archived by the owner on Oct 12, 2024. It is now read-only.

allow support of ESP32 esp-idf framework by BThome #20

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

t0urista
Copy link

@t0urista t0urista commented Oct 8, 2024

  • make compatible with ESP32 esp-idf framework by not using the pgmspace.h library when USE_ESP32 is defined
  • fixes bug in bthome_ble_receiver_hub.cpp lines 91 and 97 (payload length becomes -1, then paylod parse fails)
  • add missing object IDs in bthome_common_generated.h
  • add missiong objects IDs in const_generated.py
  • add debug capabilities in bthome_parser.cpp

avoid  pgmspace.h library
avoid pgmspace.h library
avoid pgmspace.h library
compiles without pgmspace.h library if USE_ESP32 defined
compiles without pgmspace.h library if USE_ESP32 defined
compiles without pgmspace.h library if USE_ESP32 defined
adding object IDs from  0x52 to 0x5d
adding button_hold_click 0x80
adding log in parse function
@afarago
Copy link
Owner

afarago commented Oct 12, 2024

Awesome work, really like your enthusiasm around ESP-NOW.

As stated in the page I do not maintain this repo anymore, but wanted to keep is as a reference.
Therefore I will not be testing your changes, so cannot merge the PR with an ease of mind.

I would suggest that you fork it and move the PR delta there.

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