Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Struct Without JSON Tags #809

Merged
merged 1 commit into from
Dec 10, 2023
Merged

Fix Struct Without JSON Tags #809

merged 1 commit into from
Dec 10, 2023

Conversation

chris124567
Copy link
Member

wallet.SiacoinElement values are returned by some endpoints but the type did not have struct tags. This fixes that so the encoded output conforms to our standards. I searched the code for other instances of structs with an ID field having no tags and the only other instance I found was AutopilotConfig in internal/node/node.go, but this doesn't matter because that type isn't marshaled/unmarshaled anywhere.

@peterjan peterjan merged commit 0776d92 into master Dec 10, 2023
6 checks passed
@peterjan peterjan deleted the fix-untagged-id branch December 14, 2023 11:15
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