Skip to content

Display errors in NFC scanning flow#13391

Draft
samer-stripe wants to merge 1 commit into
masterfrom
samer/improve-errors-nfc-card-reader
Draft

Display errors in NFC scanning flow#13391
samer-stripe wants to merge 1 commit into
masterfrom
samer/improve-errors-nfc-card-reader

Conversation

@samer-stripe

@samer-stripe samer-stripe commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

Display errors from NFC scanning flow under the instruction text when they are received. This text is placed under the coil without being cut off by the edges of the screen. Errors from the NfcScanningViewModel are also propagated properly to the UI while the NfcCardScanner produces a readable user message.

Motivation

Error visibility for NFC Scanning

Testing

  • Added tests
  • Modified tests
  • Manually verified

@samer-stripe samer-stripe added the nfc-scanning Work related to the NFC scanning project label Jul 10, 2026
@samer-stripe samer-stripe force-pushed the samer/improve-errors-nfc-card-reader branch from 6826eed to 1a60bf9 Compare July 11, 2026 05:11
@samer-stripe samer-stripe changed the title Improve errors for NFC scanning Display errors in NFC scanning flow Jul 11, 2026
@samer-stripe samer-stripe force-pushed the samer/improve-errors-nfc-card-reader branch from 1a60bf9 to e532f5d Compare July 11, 2026 06:53
@samer-stripe samer-stripe force-pushed the samer/improve-errors-nfc-card-reader branch from e532f5d to e108490 Compare July 11, 2026 17:33

data class Error(
val userMessage: ResolvableString,
)

@samer-stripe samer-stripe Jul 11, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will also output an error code that will be sent alongside our analytics in a follow-up.

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

Labels

nfc-scanning Work related to the NFC scanning project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant