Skip to content

Fix type on V part of signature in TransactionResponse#53

Open
timnugent wants to merge 3 commits intoCarteraMesh:mainfrom
timnugent:main
Open

Fix type on V part of signature in TransactionResponse#53
timnugent wants to merge 3 commits intoCarteraMesh:mainfrom
timnugent:main

Conversation

@timnugent
Copy link
Copy Markdown

SDK expects a string (#[serde(rename = "0")] but the response is a number ("v":0)

@timnugent
Copy link
Copy Markdown
Author

Also fixed AuthorizationGroups user parsing

@dougEfresh
Copy link
Copy Markdown
Collaborator

i included your change for generator/templates/custom/models/signed_message_signature.rs

but what is the change to src/models/authorization_groups.rs ?

Is the openapi generator creating wrong type?

  pub users: Option<std::collections::HashMap<String, Users>>,

@dougEfresh
Copy link
Copy Markdown
Collaborator

@timnugent
Looks look it's a bug with the publish spec, https://github.com/fireblocks/fireblocks-openapi-spec

dougEfresh added a commit that referenced this pull request Mar 28, 2026
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.

2 participants