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

Stop cloning transaction request instances #3750

Closed
arboleya opened this issue Mar 4, 2025 · 0 comments · Fixed by #3773
Closed

Stop cloning transaction request instances #3750

arboleya opened this issue Mar 4, 2025 · 0 comments · Fixed by #3773
Assignees
@Torres-ssf
Labels
feat Issue is a feature

Comments

@arboleya
Copy link
Member

arboleya commented Mar 4, 2025

Today, we often return a cloned transactionRequest (check below), which is not very flexible and can cause users to duplicate other code chunks if they want to customize and mutate their requests before submitting them.

fuels-ts/packages/program/src/functions/base-invocation-scope.ts

Line 246 in 86b8e94

transactionRequest = clone(transactionRequest);

cc @luizstacio
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Torres-ssf Torres-ssf

Labels
feat Issue is a feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: support customizing TX request from BaseInvocationScope FuelLabs/fuels-ts
2 participants
@arboleya @Torres-ssf