-
-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Add NFC binary sensor platform #6068
Conversation
Hey there @jesserockz, mind taking a look at this pull request as it has been labeled with an integration ( |
Hey there @jesserockz, mind taking a look at this pull request as it has been labeled with an integration ( |
Hey there @jesserockz, mind taking a look at this pull request as it has been labeled with an integration ( |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #6068 +/- ##
=======================================
Coverage 53.60% 53.60%
=======================================
Files 50 50
Lines 9374 9374
Branches 1650 1650
=======================================
Hits 5025 5025
Misses 4050 4050
Partials 299 299 ☔ View full report in Codecov by Sentry. |
42bc3f5
to
5b8cf39
Compare
5b8cf39
to
0f39f8f
Compare
What does this implement/fix?
This implements an
nfc
binary_sensor
platform for use with the PN71x0 components.In the future, other NFC components (such as the PN532 or RC522) could be adapted to use it, as well.
Types of changes
Related issue or feature (if applicable): fixes
Pull request in esphome-docs with documentation (if applicable): esphome/esphome-docs#3526
Test Environment
Example entry for
config.yaml
:Checklist:
tests/
folder).If user exposed functionality or configuration variables are added/changed: