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.

Sign up for GitHub

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

Jump to bottom

abigen: exclude first 3 fields instead of name matching #71

Closed
digorithm opened this issue Nov 30, 2021 · 0 comments · Fixed by #124
Closed

abigen: exclude first 3 fields instead of name matching #71

digorithm opened this issue Nov 30, 2021 · 0 comments · Fixed by #124
Assignees
@QuinnLee
Labels
feat

Comments

@digorithm
Copy link
Member

As discussed in FuelLabs/fuels-rs#11 (comment), once the Sway compiler removes the need to have gas, color, and coin amount as mandatory arguments to every ABI method, we should go back to this place in the fuels-ts code and remove the workaround.
@digorithm digorithm added the feat label Nov 30, 2021
@digorithm digorithm mentioned this issue Nov 30, 2021
Merged
@QuinnLee QuinnLee self-assigned this Jan 11, 2022
@QuinnLee QuinnLee changed the title Remove workaround placed to handle temporary mandatory arguments abigen: exclude first 3 fields instead of name matching Feb 7, 2022
@QuinnLee QuinnLee linked a pull request Feb 14, 2022 that will close this issue
feat!: Update interface and fragment to ignore gas, amount and color in #124
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@QuinnLee QuinnLee

Labels
feat
Projects
Fuel Network
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat!: Update interface and fragment to ignore gas, amount and color in FuelLabs/fuels-ts
2 participants
@QuinnLee @digorithm