Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added espnow module with documentation. #3644

Merged
merged 2 commits into from Mar 27, 2024
Merged

Conversation

jmattsson
Copy link
Member

Fixes #2924, at least for the ESP32 range.

  • This PR is for the dev branch rather than for the release branch.
  • This PR is compliant with the other contributing guidelines as well (if not, please describe why).
  • I have thoroughly tested my contribution.
  • The code changes are reflected in the documentation at docs/*.

This is a pretty straight-forward module implementation of the regular ESP-NOW API. Tested on ESP32-C3, but should be board agnostic.

@pjsg, quick review if you have the time, please?
@marcelstoer, could you please check I did the docs properly? It's been a while since I was active...

@marcelstoer
Copy link
Member

Here's what I found, pls verify.
0001-Docs-clean-up.patch

@jmattsson
Copy link
Member Author

Thanks Marcel!

I'll keep the small non-espnow updates out of this PR, and will probably just push them directly later 👍

@jmattsson
Copy link
Member Author

Alright, I'm making an executive decision and will merge this. I'm working on getting support for the C6 and H2 into the build, and this will collide with that so I'd rather just land this now and make life easier for everyone. Well, myself at least :D You may yell at me later if you object!

@jmattsson jmattsson merged commit bc3b31f into nodemcu:dev-esp32 Mar 27, 2024
16 checks passed
@jmattsson jmattsson deleted the espnow branch March 27, 2024 04:44
@poorandunlucky
Copy link

espnow works on esp8266, too...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants