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
Fix TAG review comments. #88
Conversation
…3c#84. Handle push related TAG review comments: simplified and aligned push message, optional push options with sensible defaults, improved push and cancelPush steps, option for suspending watches during push(), editorials.
Looks like the changes didn't update the comments here. You need to look at |
Examples need to be synchronized with latest changes. navigator.nfc.watch() for instance. |
e872e74
to
259eebd
Compare
…eb NFC Id creation.
Editors believe the TAG review comments have been addressed. |
Fix #66, #67, #68: Remove NFCAdapter. Simplify nfc namespace. Use the merged NFC adapter concept for multiple adapters. Update examples.
Fix #69, #70, #73, #77, #80, #81, #82, #83, #84. Handle push related TAG review comments: simplified and aligned push message, optional push options with sensible defaults, improved push and cancelPush steps, option for suspending watches during push(), editorials.
Fix #71, #83, #85. Handle watch related TAG review comments: fix IDL for unwatch(), reverse param order for watch() and make options optional, make cancelPush target optional.