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: cip64 regressions #6
Changes from 1 commit
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. Nit: (not related to changes in this PR) Based on this recent Slack convo with @mcortesi, transaction receipts do not include transactionReceipt: /*#__PURE__*/ defineTransactionReceipt({
format(
args: CeloRpcTransactionReceiptOverrides,
): CeloTransactionReceiptOverrides {
return {
- feeCurrency: args.feeCurrency,
- gatewayFee: args.gatewayFee ? hexToBigInt(args.gatewayFee) : null,
- gatewayFeeRecipient: args.gatewayFeeRecipient,
}
},
}), I suggest we fix this in a separate PR to keep this one clean and get the regression fix merged as soon as possible. |
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.
Nit: (not related to changes in this PR)
Based on this recent Slack convo with @mcortesi, transaction receipts do not include
feeCurrency
,gatewayFee
, orgatewayFeeRecipient
fields, which means they areundefined
in every transaction receipt object (by default). We can thus look into removing them from the expected and actual transaction receipt (in tests), because they will always trivially be equal.I suggest we fix this in a separate PR to keep this one clean and get the regression fix merged as soon as possible.