Skip to content

Conversation

@sbihel
Copy link
Member

@sbihel sbihel commented Dec 18, 2025

Description

This adds support for the new NFC functions for readers (and addresses breaking changes).

Depends on:

Other changes

  • Handle request handling a bit better, by raising warnings when verification failed and by erroring when the parsing completely failed.
  • Enable Rust logging in Android Showcase by default. I'm not sure why this wasn't already the case, but as Showcase is becoming much more of a development app, I think this will save time.
  • Close BLE connection on Android when an error occurs. This way the reader doesn't wait for a response that will never come.

Optional section

Showcase wasn't updated with NFC reader support because my end project isn't mobile-based, so it would take me some time to make it work in Android.

Tested

  1. Made sure Showcase still compiled (Android and iOS).
  2. Used the public functions in another Rust project.

I'm not sure why this wasn't already the case, but as Showcase is
becoming much more of a development app, I think this will save time.
This way the reader doesn't wait for a response that will never come
@sbihel sbihel requested review from SeaSkyThe and alichay December 18, 2025 17:52
@sbihel sbihel merged commit 2ae3b08 into main Jan 5, 2026
4 checks passed
@sbihel sbihel deleted the acc-1010-add-nfc-static-handover-reader-functions-in-sprucekit-mobile branch January 5, 2026 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants