-
Notifications
You must be signed in to change notification settings - Fork 122
feat: Add exclude_function_args parameter to reduce transaction response sizes #2312
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
Changes from 11 commits
c5cfc5a
6abae65
4c26296
985a147
d320d87
2273bf4
737b7d9
58014ad
f7a6b19
14bb7c3
97aca16
4567592
b5ac1ac
d6e1781
7dd5048
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Apologies, I should've explained better how this file works. This file is edited by Can you revert this file to how it was before? You can do that with
Sorry for the confusion on this! |
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -864,6 +864,7 @@ export async function getRosettaTransactionFromDataStore( | |||||
interface GetTxArgs { | ||||||
txId: string; | ||||||
includeUnanchored: boolean; | ||||||
excludeFunctionArgs?: boolean; | ||||||
|
excludeFunctionArgs?: boolean; | |
excludeFunctionArgs: boolean; |
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make these two new args required instead of optional. Set to false whenever you don't have it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
excludeFunctionArgs?: boolean; | |
excludeFunctionArgs: boolean; |
Uh oh!
There was an error while loading. Please reload this page.