Re: [web-nfc] Change `toRecords()` to `records` (#362)

It's fine to return `null`.
However, length check requires that the implementation checks whether the record could be parsed as a message before the user actually attempts that. 
For `"Sp"` that is fine since its a well known type. 
However, for external and unknown it is not fine. The best would be to have a method that can be invoked when the app knows this should succeed (since they have written the other end, for instance).

-- 
GitHub Notification of comment by zolkis
Please view or discuss this issue at https://github.com/w3c/web-nfc/issues/362#issuecomment-538002769 using your GitHub account

Received on Thursday, 3 October 2019 15:44:26 UTC