Skip to content

fix(nfc): adapt backend invoke to frontend - #3419

Merged
FabianLars merged 6 commits into
tauri-apps:v2from
RoBoa-Robotics:fix/nfc-backend-return-type
Jul 20, 2026
Merged

FabianLars merged 6 commits into
tauri-apps:v2from
RoBoa-Robotics:fix/nfc-backend-return-type

Conversation

@pascalauroboa

Copy link
Copy Markdown
Contributor

Should resolve #2272. There is a mismatch between the frontend and backend API, as well as the native implementations. Javascript works but Rust does not. This should change the API to match the working frontend.

@pascalauroboa
pascalauroboa requested a review from a team as a code owner May 11, 2026 21:11
@pascalauroboa
pascalauroboa force-pushed the fix/nfc-backend-return-type branch from ff314fe to e34306e Compare May 11, 2026 21:20
@github-actions

github-actions Bot commented May 11, 2026 •

Copy link
Copy Markdown
Contributor

Package Changes Through 2baf630

There are 2 changes which include nfc with patch, nfc-js with patch

Planned Package Versions

The following package releases are the planned based on the context of changes in this pull request.

package current next
api-example 2.0.46 2.0.47
api-example-js 2.0.42 2.0.43
nfc 2.3.5 2.3.6
nfc-js 2.3.5 2.3.6

Add another change file through the GitHub UI by following this link.


Read about change files or the docs at github.com/jbolda/covector

@FabianLars

Copy link
Copy Markdown
Member

thanks for the PR! Unfortunately, changing the function signature and remove a pub struct are both breaking changes so this has to wait for v3. until then, does this work?

    pub fn scan(&self, payload: ScanRequest) -> crate::Result<ScanResponse> {
        self.0
            .run_mobile_plugin("scan", payload)
            .map(|v| ScanResponse { tag: v })
            .map_err(Into::into)
    }

Lastly, please also add a changefile similar to https://raw.githubusercontent.com/tauri-apps/plugins-workspace/refs/heads/v2/.changes/material-dialog.md , thanks :)

Comment thread .changes/nfc.md Resolved
Co-authored-by: Fabian-Lars <30730186+FabianLars@users.noreply.github.com>
@FabianLars
FabianLars merged commit 5267261 into tauri-apps:v2 Jul 20, 2026
12 checks passed
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.

Invoking an NFC scan from Rust causes serialization errors

2 participants